? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
9 Jan 2020

When you assign a module to a menu item there is a select for On All Pages, Selected etc

image

If you are on the either of the two "selected" options you will see a search box and a tree.

  1. The search input does not have a label - for a11y it must have one - this PR adds aria-label attribute to resolve that

  2. The search input has tabindex-1 which means that it is not part of the navigation tree and you can never tab into the input to perform a search - this PR removes the tabindex so you can now tab into the search input

avatar brianteeman brianteeman - open - 9 Jan 2020
avatar brianteeman brianteeman - change - 9 Jan 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Jan 2020
Category Administration com_menus com_modules
avatar brianteeman brianteeman - change - 9 Jan 2020
Labels Added: ?
avatar Quy Quy - test_item - 9 Jan 2020 - Tested successfully
avatar Quy
Quy - comment - 9 Jan 2020

I have tested this item successfully on 3506d83


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

avatar jwaisner jwaisner - test_item - 9 Jan 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 9 Jan 2020

I have tested this item successfully on 3506d83


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

avatar Quy Quy - change - 9 Jan 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 9 Jan 2020

RTC


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

avatar Quy Quy - change - 9 Jan 2020
Labels Added: ?
avatar wilsonge wilsonge - change - 11 Jan 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-01-11 13:36:00
Closed_By wilsonge
avatar wilsonge wilsonge - close - 11 Jan 2020
avatar wilsonge wilsonge - merge - 11 Jan 2020
avatar wilsonge
wilsonge - comment - 11 Jan 2020

Thanks!

avatar brianteeman
brianteeman - comment - 11 Jan 2020

w00t

Add a Comment

Login with GitHub to post a comment