? ? 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

screenshotr14-48-15

Thanks to @fuzzbomb for his guidance on this - there is more to come

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 Administration com_associations com_categories com_contact com_content com_menus com_modules com_newsfeeds
0c7e658 17 Mar 2017 avatar brianteeman file
avatar brianteeman brianteeman - change - 17 Mar 2017
Labels Added: ?
avatar laoneo laoneo - test_item - 20 Mar 2017 - Tested successfully
avatar laoneo
laoneo - comment - 20 Mar 2017

I have tested this item successfully on 9df52ef

Didn't test it with an accessibility reader, but according to the docs its the right way.


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

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

I have tested this item successfully on 9df52ef


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

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/14689.

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

Thanks merging!

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:00
Closed_By zero-24
Labels Added: ?
avatar zero-24 zero-24 - close - 29 Mar 2017
avatar zero-24 zero-24 - merge - 29 Mar 2017
avatar brianteeman
brianteeman - comment - 29 Mar 2017

Thanks for testing and merging

Add a Comment

Login with GitHub to post a comment