? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
4 Oct 2018

Pull Request for Issue # .

Summary of Changes

This replaces link tags that look and act like buttons with actual buttons. See #20682 for details.

Testing Instructions

Code review.

avatar SharkyKZ SharkyKZ - open - 4 Oct 2018
avatar SharkyKZ SharkyKZ - change - 4 Oct 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Oct 2018
Category Administration com_banners com_categories com_contact com_content com_fields com_menus com_modules com_newsfeeds com_redirect com_tags com_templates com_users
avatar brianteeman
brianteeman - comment - 4 Oct 2018

I have tested this item successfully on 6401c60

Code Review


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

avatar brianteeman brianteeman - test_item - 4 Oct 2018 - Tested successfully
avatar SharkyKZ SharkyKZ - change - 4 Oct 2018
Labels Added: ?
avatar Quy
Quy - comment - 4 Oct 2018

I have tested this item successfully on e65cff5


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

avatar Quy Quy - test_item - 4 Oct 2018 - Tested successfully
avatar Quy Quy - change - 4 Oct 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 4 Oct 2018

RTC


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

avatar mbabker mbabker - change - 6 Oct 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-10-06 18:48:25
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 6 Oct 2018
avatar mbabker mbabker - merge - 6 Oct 2018
avatar brianteeman
brianteeman - comment - 6 Oct 2018

woohoo

avatar brianteeman
brianteeman - comment - 6 Oct 2018

@SharkyKZ would you like to do a similar pr for J4?

avatar SharkyKZ
SharkyKZ - comment - 6 Oct 2018

I thought everything that goes into 3.x is supposed to reach 4.0 eventually. I can see a lot of a11y stuff going into 4.0 now. But, IMO, simple markup fixes like this should be done against 3.x too.

avatar brianteeman
brianteeman - comment - 6 Oct 2018

you are correct but we have a really big backlog on that and I would really like to get as much a11y stuff done in J4 as quickly as possible. Plus I suspect that a lot of the remaining links->buttons are modal related and we plan to change from a bootstrap modal at some point so it would be good to have them changed now as it will be hard to add later from the 3.x branch

avatar SharkyKZ
SharkyKZ - comment - 8 Oct 2018

Should this be done for both 3.x and 4.0 then? Or just 4.0?

Add a Comment

Login with GitHub to post a comment