On 08/09/2017 06:28 PM, Stuart Marks wrote:
On 8/9/17 5:11 PM, Jonathan Gibbons wrote:
Please review this proposed change to address accessibility and HTML issues
in the java.base java.lang[.*] 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/8186052/webrev.00/index.html
API: http://cr.openjdk.java.net/~jjg/8186052/api.00/index.html

Hi Jon,

These changes look good. In most cases the table surgery was a big improvement. One small quibble I have is with the table in String.split(regex, limit). The limit column has numbers, and the alignment changed from centered to left-aligned, neither of which is really very good. I'd suggest changing this column to be right-aligned, so the negative values in this column will be aligned appropriately.

Thanks,

s'marks

Stuart,

Thanks for the suggestion. I'll fix that.

-- Jon

Reply via email to