? ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
27 Jun 2019

As a result of the accessibility performed last night on the Permissions page it was realised that it would be best for screen reader users to know what the changed setting is. For example if they chose "inherited" they need to know that this means that "allowed - inherited" has been selected.

The super easy way to do this is to wrap the "allowed - inherited" span in the html element output. This way whenever the output is updated it is automatically announced by the screen reader.

Technically we could remove the span and put the class on the <output> element but as that would require changing the javascript as well I didn;t see the need to do that. The code in this PR works perfectly without making that change

Pull Request for Issue # .

Testing Instructions

Code review or by testing with a screen reader

avatar brianteeman brianteeman - open - 27 Jun 2019
avatar brianteeman brianteeman - change - 27 Jun 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Jun 2019
Category Layout
avatar brianteeman brianteeman - change - 12 Jul 2019
Labels Added: ? ?
avatar tonypartridge
tonypartridge - comment - 13 Jul 2019

I have tested this item successfully on 7a34245


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

avatar tonypartridge tonypartridge - test_item - 13 Jul 2019 - Tested successfully
avatar alikon alikon - test_item - 15 Jul 2019 - Tested successfully
avatar alikon
alikon - comment - 15 Jul 2019

I have tested this item successfully on 7a34245

code review


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

avatar alikon alikon - change - 15 Jul 2019
Status Pending Ready to Commit
avatar alikon
alikon - comment - 15 Jul 2019

RTC


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

avatar wilsonge wilsonge - change - 23 Jul 2019
Labels Added: ?
avatar wilsonge wilsonge - change - 23 Jul 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-23 18:05:48
Closed_By wilsonge
avatar wilsonge wilsonge - close - 23 Jul 2019
avatar wilsonge wilsonge - merge - 23 Jul 2019
avatar wilsonge
wilsonge - comment - 23 Jul 2019

Thanks!

avatar brianteeman
brianteeman - comment - 23 Jul 2019

thanks

Add a Comment

Login with GitHub to post a comment