? ? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
23 Jul 2019

Pull Request for Issue #25642 (comment) and #25642 (comment).

Summary of Changes

Change HTML element of the extension name from <label> to <span> in the extensions manager's list view because the check box to select the item is already a label for the same thing, and duplicate labels for same thing is invalid HTML.

See also the comment mentioned above as issue.

Testing Instructions

Inspect with your browser's developer tools the HTML of the extensions manager's list view.

Check the label element just before the check box and then the extension name for a particular extension.

Or validate the page and check for duplicate label errors.

Expected result

Only 1 label for the check box, which is the element before the check box, and the extension name is not another label for that.

Actual result

1 label for the check box, which is the element before the check box, and the extension name is another label for that.

Documentation Changes Required

None.

avatar richard67 richard67 - open - 23 Jul 2019
avatar richard67 richard67 - change - 23 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jul 2019
Category Administration com_installer
avatar richard67 richard67 - change - 23 Jul 2019
The description was changed
avatar richard67 richard67 - edited - 23 Jul 2019
avatar richard67
richard67 - comment - 23 Jul 2019

@Quy Done.

avatar richard67 richard67 - change - 23 Jul 2019
Labels Added: ?
avatar Quy
Quy - comment - 23 Jul 2019

I have tested this item successfully on 2c8090c


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

avatar Quy Quy - test_item - 23 Jul 2019 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 25 Jul 2019

I have tested this item successfully on 2c8090c

Should be correct with: <span tabindex="0">Banners
OT: I had to enter the verification code.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25696.
avatar ChristineWk
ChristineWk - comment - 25 Jul 2019

I have tested this item successfully on 2c8090c

Should be correct with: <span tabindex="0">Banners
OT: I had to enter the verification code.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25696.
avatar ChristineWk ChristineWk - test_item - 25 Jul 2019 - Tested successfully
avatar Quy Quy - change - 25 Jul 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 25 Jul 2019

RTC


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

avatar Quy
Quy - comment - 25 Jul 2019

RTC


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

avatar wilsonge wilsonge - change - 27 Jul 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-27 20:01:03
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 27 Jul 2019
avatar wilsonge wilsonge - merge - 27 Jul 2019
avatar wilsonge
wilsonge - comment - 27 Jul 2019

Thanks!

avatar richard67
richard67 - comment - 27 Jul 2019

Thanks.

Add a Comment

Login with GitHub to post a comment