Hi Priya, Looks good to me.
It’s a bit sad that we lose the possibility to make the style customizable and have to hard code it in HTML, but I think there’s no way around that, and the cleaner HTML structure is probably worth it. Hannes > Am 25.04.2019 um 08:52 schrieb Priya Lakshmi Muthuswamy > <[email protected]>: > > Hi, > > Kindly review the fix for https://bugs.openjdk.java.net/browse/JDK-8221991 > > webrev : http://cr.openjdk.java.net/~pmuthuswamy/8221991/webrev.00/ > structure:http://cr.openjdk.java.net/~pmuthuswamy/8221991/structure/HashMap.html > > Existing Structure: > <ul class="inheritance"> > <li><a href="../lang/Object.html" title="class in > java.lang">java.lang.Object</a></li> > <li> > <ul class="inheritance"> > <li><a href="AbstractMap.html" title="class in > java.util">java.util.AbstractMap</a><K,V></li> > <li> > <ul class="inheritance"> > <li>java.util.HashMap<K,V></li> > </ul> > </li> > </ul> > </li> > </ul> > Current Structure : > <ul class="inheritance" title="Inheritance Tree"> > <li><a href="../lang/Object.html" title="class in > java.lang">java.lang.Object</a></li> > <li style="margin-left:2em"><a href="AbstractMap.html" title="class in > java.util">java.util.AbstractMap</a><K,V></li> > <li style="margin-left:4em">java.util.HashMap<K,V></li> > </ul> > > Thanks, > Priya >
