User tests: Successful: 2 Quy, tonypartridge Unsuccessful: 0
When reading our default markup for rendering icons, assisistive technology may have the following problems.
The assistive technology will not find any content to read out to a user
The assistive technology will read the unicode equivalent, which does not match up to what the icon means in context, or worse is just plain confusing. In our use case it is always plain wrong. For example the unicode character used to display the trashed icon is \4c which is equal to L
When an icon is not an interactive element the simplest way to provide a text alternative is to use the aria-hidden="true" attribute on the icon and to include the text with an additional element, such as a < span>, with appropriate CSS to visually hide the element while keeping it accessible to assistive technologies.
In this case we have the text and it is displayed so we just need to add the aria-hidden to prevent the icon being "read aloud"
This PR should address all the remaining icons in the templates folder where we do have text
Status | New | ⇒ | Pending |
Category | ⇒ | Front End Templates (site) |
Category | Front End Templates (site) | ⇒ | Accessibility Front End Templates (site) |
I have tested this item
Code review
I have tested this item
Code Review.
Drone did rerun and is happy now
Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-24 20:06:12 |
Closed_By | ⇒ | rdeutz | |
Labels |
Added:
?
|
Thanks
No idea why this failed appveyor test. looks like an issue with appveyor not this pr