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 >
