? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
21 Mar 2021

Does it go somewhere? It's a link
Does it do something? It's a button

This PR corrects the usage of an a when it should be a button in the non-table repeatable layouts. You can see this in action in the CSP options after selecting custom mode and scrolling down the page.

There is no visible or functional change.

Note the javascript can probably be updated to not listen for a spacebar key as that is native for a button - this is beyond the scope of this PR.

PS I already did this once before in #26614

avatar brianteeman brianteeman - open - 21 Mar 2021
avatar brianteeman brianteeman - change - 21 Mar 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Mar 2021
Category Layout
avatar brianteeman brianteeman - change - 21 Mar 2021
The description was changed
avatar brianteeman brianteeman - edited - 21 Mar 2021
avatar brianteeman brianteeman - change - 21 Mar 2021
The description was changed
avatar brianteeman brianteeman - edited - 21 Mar 2021
avatar Fedik Fedik - test_item - 21 Mar 2021 - Tested successfully
avatar Fedik
Fedik - comment - 21 Mar 2021

I have tested this item successfully on 31fade9


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

avatar Quy Quy - test_item - 21 Mar 2021 - Tested successfully
avatar Quy
Quy - comment - 21 Mar 2021

I have tested this item successfully on 31fade9


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

avatar Quy Quy - change - 21 Mar 2021
Status Pending Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 21 Mar 2021

RTC


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

avatar wilsonge wilsonge - change - 21 Mar 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-03-21 23:44:12
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 21 Mar 2021
avatar wilsonge wilsonge - merge - 21 Mar 2021
avatar wilsonge
wilsonge - comment - 21 Mar 2021

Thanks!

avatar brianteeman
brianteeman - comment - 22 Mar 2021

thanks

Add a Comment

Login with GitHub to post a comment