Skip to content

Improve a11y (closes #1206)#1665

Merged
aristov merged 1 commit intov2from
issues/1206-common@v2
Sep 29, 2015
Merged

Improve a11y (closes #1206)#1665
aristov merged 1 commit intov2from
issues/1206-common@v2

Conversation

@aristov
Copy link
Contributor

@aristov aristov commented Sep 8, 2015

Closes #1206.

@aristov
Copy link
Contributor Author

aristov commented Sep 22, 2015

Known issues:

role="presentation"
JAWS works incorrect with the role="presentation" and ignores it for sometimes.
E.g. it reads label for radio or checkbox twice despite a presence of the "presentation" role.

role="group"
JAWS ignores the "group" role without the title/aria-label/aria-labelledby attribute. This applies to the control-group, checkbox-group and menu__group.

radio
Using the "Spacebar" key for switching radio buttons causes wrong behaviour of JAWS.
It works properly only when the "Enter" + arrow keys are used for switching.

radio_type_button
JAWS still reads it like a list of toggle buttons.
The attempt to fix it still doesn't work properly, but already looks like an ugly workaround.
One of the solution may be a reconsideration of the architectural approach, where button will be used for styling only, but not for the behaviour management goals.

menu
"Spacebar" key triggers a flash of the hovered state on the first menu-item and JAWS speaks it.
On the first "Down arrow" keypress VPC comes to the same item and JAWS doesn't speak the same for the second time.

select_mode_check
JAWS works incorrect with the aria-owns attribute.
This causes an unstable behaviour for the "Extended select listbox".

popup and modal
Focus management problems will be solved later and separately from this PR.

attach
JAWS speaks attach twice and we can not fix it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO add comment about workaround here

@aristov aristov force-pushed the issues/1206-common@v2 branch from e34ba3c to 4942fce Compare September 29, 2015 08:07
@aristov
Copy link
Contributor Author

aristov commented Sep 29, 2015

TODO fix gemini tests, squash and merge.

@aristov aristov force-pushed the issues/1206-common@v2 branch from 702188e to db4a138 Compare September 29, 2015 12:35
@aristov
Copy link
Contributor Author

aristov commented Sep 29, 2015

Squashed.

aristov pushed a commit that referenced this pull request Sep 29, 2015
@aristov aristov merged commit 5406834 into v2 Sep 29, 2015
@aristov aristov removed the ready label Sep 29, 2015
@aristov aristov deleted the issues/1206-common@v2 branch September 29, 2015 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant