? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
17 Mar 2017

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 addresses the use case shown in the image below
screenshotr15-09-11

avatar brianteeman brianteeman - open - 17 Mar 2017
avatar brianteeman brianteeman - change - 17 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Mar 2017
Category Modules Administration Front End com_content Layout
avatar laoneo laoneo - test_item - 20 Mar 2017 - Tested successfully
avatar laoneo
laoneo - comment - 20 Mar 2017

I have tested this item successfully on d698b42

Didn't test it with an accessibility reader, but according to the docs its the right way. Nice that Joomla is taking care about all.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14690.

avatar alikon alikon - test_item - 28 Mar 2017 - Tested successfully
avatar alikon
alikon - comment - 28 Mar 2017

I have tested this item successfully on d698b42


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14690.

avatar jeckodevelopment jeckodevelopment - change - 28 Mar 2017
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 28 Mar 2017

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14690.

avatar zero-24
zero-24 - comment - 29 Mar 2017

merging 👍

avatar zero-24 zero-24 - close - 29 Mar 2017
avatar zero-24 zero-24 - merge - 29 Mar 2017
avatar zero-24 zero-24 - change - 29 Mar 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-29 11:08:17
Closed_By zero-24
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 29 Mar 2017

Thanks for mergin

Add a Comment

Login with GitHub to post a comment