OK,
It would be better if the placement were consistent, but this is a step
in the right direction.
Looking at the use of empty comments, it seems fundamentally flawed,
which is part of the white-space issue you were seeing, but that's a
separate issue that we can handle separately.
This is OK to push.
-- Jon
On 05/27/2019 04:04 PM, Jonathan Gibbons wrote:
Ah, OK, I missed that in the webrev; I'll take another look.
-- Jon
On 5/26/19 9:24 PM, Priya Lakshmi Muthuswamy wrote:
Hi Jon,
The extra blank space was appearing if we place anchor before the the
heading.
This issue mainly occurs in field/method/constructor... detail section.
It is addressed by placing anchor within the header element.
Example of field details(separatorChar) in File.html :
<section class="detail">
<h3><a id="separatorChar">separatorChar</a></h3>
In the summary section, blank space issue is not there, so have
placed anchor before the heading.
File.html structure:
http://cr.openjdk.java.net/~pmuthuswamy/8223364/8223364/review.01/File.html
Thanks,
Priya
On 5/24/2019 9:49 PM, Jonathan Gibbons wrote:
Hi Priya,
Previously, you were concerned about extra blank space appearing
above the heading.
Can you point to what you have done to address that issue?
-- Jon
On 5/24/19 5:41 AM, Priya Lakshmi Muthuswamy wrote:
Hi,
Kindly review the fix for
https://bugs.openjdk.java.net/browse/JDK-8223364
webrev :
http://cr.openjdk.java.net/~pmuthuswamy/8223364/8223364/review.01/webrev.00/
api :
http://cr.openjdk.java.net/~pmuthuswamy/8223364/8223364/review.01/api/
Thanks,
Priya
