NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
21 Apr 2019

Fix the tabbing of modals in media manager. Before the PR tabs are totally locked on the input element in both the create folder and the rename modal. After this PR the focus in all modals is locked to the modal. However all elements in that (buttons etc) can be tabbed into.

avatar wilsonge wilsonge - open - 21 Apr 2019
avatar wilsonge wilsonge - change - 21 Apr 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Apr 2019
Category Administration com_media NPM Change JavaScript
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Apr 2019
Title
[4.0] [a11y] Fix modals tabbing in media manager
[4.0] Fix modals tabbing in media manager
avatar franz-wohlkoenig franz-wohlkoenig - edited - 21 Apr 2019
avatar brianteeman
brianteeman - comment - 21 Apr 2019

Great will put it on todays test list

avatar richard67
richard67 - comment - 21 Apr 2019

I have tested this item successfully on 9949e5e

1. Checked modals for file rename and file creation in media manager.
2. Applied patch with patchtester.
3. Did composer install and nmp install.
4. Checked modals for file rename and file creation in media manager.

Result:
Before patch no tabbing inside modals.
After patch tabbing inside modals is possible.


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

avatar richard67 richard67 - test_item - 21 Apr 2019 - Tested successfully
avatar brianteeman
brianteeman - comment - 21 Apr 2019

I have tested this item successfully on 9949e5e


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

avatar brianteeman brianteeman - test_item - 21 Apr 2019 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Apr 2019
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 21 Apr 2019

Status "Ready To Commit".

avatar wilsonge wilsonge - change - 21 Apr 2019
Labels Added: ?
avatar wilsonge wilsonge - close - 21 Apr 2019
avatar wilsonge wilsonge - merge - 21 Apr 2019
avatar wilsonge wilsonge - change - 21 Apr 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-04-21 17:30:21
Closed_By wilsonge
Labels Added: NPM Resource Changed ?
avatar wilsonge
wilsonge - comment - 21 Apr 2019

Thanks guys!

avatar richard67
richard67 - comment - 21 Apr 2019

Thanks too.

Add a Comment

Login with GitHub to post a comment