Thanks.
I think we should review the use of all the values in the Role enum, to
see whether they are necessary and/or being used correctly according to
the latest documents. For example, I see lots of `<nav
role="navigation">` and `<main role="main">` which might both be
questionable, and worth investigating.
-- Jon
On 4/5/19 5:45 AM, Hannes Wallnöfer wrote:
Looks good.
Should we remove REGION from the Role enum as it isn’t used anymore?
Hannes
Am 02.04.2019 um 23:13 schrieb Jonathan Gibbons <[email protected]>:
Please review a small change to bring javadoc in line with the HTML 5 spec.
The HTML5 spec advises that `role="region"` should not be set on <section>
elements.
Allowed ARIA role attribute values:
region role (default - do not set), alert, alertdialog, application,
contentinfo , dialog, document, feed, log, main, marquee, presentation, region,
search, status or tabpanel.
The attribute was being set unconditionally. The lines to do this have been
removed and the necessary tests updated.
-- Jon
Spec: https://www.w3.org/TR/html5/sections.html#the-section-element
JBS: https://bugs.openjdk.java.net/browse/JDK-8221871
Webrev: http://cr.openjdk.java.net/~jjg/8221871/webrev.00/index.html
