[PATCH][CCLA] dijit.popup functionality needs aria roles and attributes

[mikeb]

Anything that uses popup functionality from _popup.js needs to have aria-label and aria-role attributes. This is because according to WAI-ARIA, all content needs to be enclosed in either "page landmarks" or in a role=region. The problem is that _popup.js will create a "wrapper" for the popup dijit and appends this domNode to the end of the dom tree. This means that the popup dijit's contents will actually not be enclosed in a role=region or landmark unless the author has set the role attribute on the body tag. Not everybody can or should set role=main on the body element.

The patch takes care of automatically setting role="region" on the wrapper of the domNode. It also has to set aria-label attribute since role=region requires that attribute. I had some trouble trying to figure out what the best way to set the aria-label was, so I just look for various attributes ("aria-label", "label", "name") and eventually fall back on the ID.

I tested with JAWS14 and there is no difference in behavior with this patch.

I also created some automated test cases and verified that the test pages all still work. I think I hit every dijit that uses _popup.

[mikeb]

set role=region and aria-label on the wrapper element of any popup, plus tests, please proxy commit for michael billau ccla on file with ibm.

[bill]

OK thanks. I needed to modify most of the patch to use modern AMD style (i.e. no reference to the dojo global) and to fix spacing, but I'll check in.

[bill]

In [30213]:

Set role=region for the popup wrapper. Fixes #16466 !strict. Patch from Mike Billau (IBM, CCLA), thanks!

[Douglas Hays]

Can this go into 1.8 as well?

[bill]

I guess so, doesn't look like it will break anything.

[Douglas Hays]

In [30317]:

Refs #16466. Backport [30213] to 1.8. !strict