RFR: 8184021: Fix tables in jaxp doc comments to be accessible
Lance Andersen
lance.andersen at oracle.com
Fri Jul 7 20:33:55 UTC 2017
Hi Jon,
The changes looked good to me. Is the html available by chance to do an extra sanity check?
Best
Lance
> On Jul 7, 2017, at 2:14 PM, Jonathan Gibbons <jonathan.gibbons at oracle.com> wrote:
>
> Please review the following fixes to the doc comments in the jaxp repo.
>
> The primary goal is to fix user-written tables in doc comments for accessibility, either by updating tables to use scope=row|col in header cells for simple tables, or by using the headings attribute in more complex tables. The majority of the changes, to tables using class="striped", were done mechanically, using a custom Java utility. Three tables were fixed up manually.
>
> In addition, I removed the use of <p> tags which introduced empty paragraph. These tags caused warnings when checking the code with the "tidy" program.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8184021
> Webrev: http://cr.openjdk.java.net/~jjg/8184021/webrev.00/
>
> -----
>
> The following list of comments about the changes is ordered according to the list in the webrev.
>
> Catalog:
> removed empty <p>
>
> CatalogFeatures:
> first table updated manually (class="plain")
> removed empty <p>
> second table updated with utility
>
> CatalogResolver:
> removed empty <p>
>
> DatatypeFactory, Duration:
> table updated with utility
>
> XMLGregorianCalendar:
> all tables updated with utility
>
> datatype/package-info:
> table updated with utility
>
> NamespaceContext:
> all tables updated with utility
>
> XMLEventWriter, XMLInputFactory, XMLOutputFactory, XMLStreamReader:
> table updated with utility
>
> XMLStreamWriter:
> this is the one table with class="striped" that had to be updated
> manually, because of the complex headers
>
> SchemaFactory:
> table updated with utility, but table organization could be improved (by spec owner)
> for example, to split first column into a name and a URI, and/or use the second
> column as a row header.
>
> Validator, XPath, XPathExpression, xpath/package-info:
> table updated manually (class="plain")
>
> Attr
> table updated with utility
> table could be improved by removing redundant atributes, like rowspan='1' colspan='1'
>
> Document, Node:
> all tables updated with utility
>
> -- Jon
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev
mailing list