Brad,
Thanks. I should have noted that the issues were detected by checking tools,
and that after the edits, no such issues were reported in this package.
FWIW, there is one remaining issue to be addressed: it is a broken link
in Provider.html.
It comes from Provider.replace(Object, Object, Object) which overrides
Properties.replace(Object, Object, Object), but for some reason, that
method is
marked @hidden in Properties, causing a broken link in Provider. That needs
to be investigated, but I don't want to block these accessibility fixes.
-- Jon
On 08/11/2017 05:14 PM, Bradford Wetmore wrote:
Changes look good to me, although I'm not an HTML expert nor an expert
in our current documentation style. I did compare the output, and can
see the effects. (Striped, scope (row vs col), style)
Brad
On 8/11/2017 5:01 PM, Jonathan Gibbons wrote:
Please review the following small fix for some accessibility issues
in the java.security package.
3 tables are converted to the de-facto JDK standard for row-oriented
tables, and updated
with appropriate scope=row|col attributes to identify the header cell
in each row or column.
You can see the appearance of the updated tables in the API link below.
This is part of the ongoing effort to clean up all such issues in the
JDK documentation.
JBS: https://bugs.openjdk.java.net/browse/JDK-8186160
Webrev: http://cr.openjdk.java.net/~jjg/8186160/webrev.00
API: http://cr.openjdk.java.net/~jjg/8186160/api.00
-- Jon
