Ooops ... forgot the updated links:
Webrev: http://cr.openjdk.java.net/~jjg/8185994/webrev.01/index.html
API: http://cr.openjdk.java.net/~jjg/8185994/api.01/index.html
-- Jon
On 08/10/2017 01:59 PM, Jonathan Gibbons wrote:
I have posted an update to the webrev and API.
The change is in DataInput ... I have tweaked the position of the text
in the cells of that initial table.
I would prefer not to iterate further on the layout of this table,
since the changeset is primarily
about addressing accessibility and HTML correctness issues.
-- Jon
On 08/10/2017 01:00 PM, Brian Burkhalter wrote:
On Aug 9, 2017, at 6:56 PM, Stuart Marks <[email protected]
<mailto:[email protected]>> wrote:
On 8/9/17 5:14 PM, Jonathan Gibbons wrote:
Please review this proposed change to address accessibility and
HTML issues
in the java.base java.io and java.nio[.*] packages.
In general, the changes consist of
* update tables to add scope=row|col as appropriate.
In some cases, slightly more surgery was needed on the table
* fix issues reported by tidy
In most cases, this is removing superfluous <p> elements.
Webrev:http://cr.openjdk.java.net/~jjg/8185994/webrev.00/index.html
<http://cr.openjdk.java.net/%7Ejjg/8185994/webrev.00/index.html>
API:http://cr.openjdk.java.net/~jjg/8185994/api.00/index.html
<http://cr.openjdk.java.net/%7Ejjg/8185994/api.00/index.html>
Again, changes mostly look good. One small formatting point on
DataInput. The numbers in the table should all have similar
alignment. Most of the numbers in the table body look centered. I
think it would look better if the numbers in the "Bit Values" column
headers, and the numbers in the Byte column, were all centered.
Since they're all single digits, you can probably get away with
centering (unless you want to change them all to right-aligned with
padding, or something).
I concur with Stuart that this is the only problem area. Centering
the Byte column would be good for sure. Centering the “Bit Values”
entries “bits x-y” could work if they were aligned in a visually
appealing manner, e.g., centered with respect to the same vertical.
Another possibility would be to have a separate cell for each
row-column position and put “b6”, “b5”, etc. in the cells perhaps
with a footnote below the table about the notation unless it’s deemed
obvious (maybe use subscript elements b<sub>6</sub>, etc.?).
Brian
