]> cat aescling's git repositories - mastodon.git/commit
Improve aria support - Columns and Navigation Drawer Icons (#2299)
authorStephen Burgess <stephenburgess8@gmail.com>
Sat, 22 Apr 2017 15:30:35 +0000 (10:30 -0500)
committerEugen <eugen@zeonfederated.com>
Sat, 22 Apr 2017 15:30:35 +0000 (17:30 +0200)
commit4cfc15556016c886aa0f385ea664680647e31aca
tree918b102b44698e6c847540eddbd1c31b519ef660
parenta1174a6d7e60dd73ab97da08b52dd751c45e2b6b
Improve aria support - Columns and Navigation Drawer Icons (#2299)

* feat(aria): Add aria-labels to underlabelled tab nav items

The drawer tabs which control primary navigation are only labelled by a title which is not available to many screenreaders. Add an aria-label attribute to each link to improve readability with screenreaders. Organize link attributes so link target is first followed by classname.
Issue #1349

* feat(aria): Replace abstract aria role of section with region

Abstract aria roles such as section should not be used in content. Use non-abstract 'region' aria role instead. That role expects an aria-labelledby attribute with an id. Pass an ID to the column header. Remove the aria-label attribute on the ColumnHeader because the same value is output in plaintext as its child.
Issue #1349

* fix(aria): Remove aria-controls attribute until solution is found

Columns do not have wrappers, so these icons can't point to a column wrapper which it controls. Instead these icons function as triggers to show or hide individual columns.
#1349

* fix(typo): Remove type of aria-labelledby instead of aria-label
app/assets/javascripts/components/features/compose/index.jsx
app/assets/javascripts/components/features/ui/components/column.jsx
app/assets/javascripts/components/features/ui/components/column_header.jsx