RFR: JDK-8186153, , Fix a11y and HTML issues in the java.math, java.text and java.time packages

[Jonathan Gibbons]

Lance,

I was in two minds about changing the layout of the columns in RoundingMode.
It can sometimes hard to discern the intent of the author. In addition, 
numbers
can be problematic because ideally you want the overall column of numbers to
"centered", but each individual number to be right-aligned within that 
centered area
-- and that is not easily expressible.  The best that we've achieved 
elsewhere is to use
"right-aligned, with some right padding". While that can be done, the local
style declarations start overwhelming the content ... and the whole trend of
HTML5 and CSS is to separate content and style.

In this case, there are several tables that would benefit from a 
specific style
declaration, but it's not important enough IMO to add to the main 
javadoc stylesheet.
But, the javadoc team is working on enhancements to the use of stylesheets,
and maybe that will give a better way forward, eventually.

But I agree the left-aligned values look less than ideal. The question 
is, do you
want to add one of the following to all the appropriate cells:
     style="text-align:center"
     style="text-align:right; padding-right=Nem"  // for some value of N

-- Jon


On 08/11/2017 11:44 AM, Lance Andersen wrote:
> Hi Jon,
>
> Looks OK overall
>
> Any thoughts on centering some of the column values such as in 
> RoundingMode at some point?
>
>
> DateTimeFormatter looks much nicer :-)
>
> Best
> Lance
>
>
>> On Aug 11, 2017, at 2:28 PM, Jonathan Gibbons 
>> <jonathan.gibbons at oracle.com <mailto:jonathan.gibbons at oracle.com>> wrote:
>>
>> Please review the following changes to fix accessibility and HTML 
>> issues in the
>> java.math, java.text and java.time packages.
>>
>> Mostly, it's "more of the same", setting scope=row|col on tables.
>>
>> One bigger change is in DateTimeFormatter, to change a preformatted ASCII
>> table into a standard HTML table.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8186153
>> Webrev: http://cr.openjdk.java.net/~jjg/8186153/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Ejjg/8186153/webrev.00/>
>> API: http://cr.openjdk.java.net/~jjg/8186153/api.00/ 
>> <http://cr.openjdk.java.net/%7Ejjg/8186153/api.00/>
>>
>> -- 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>
>
>
>