? ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
15 May 2018

Fix regression for Issue #19969.

Summary of Changes

PR #19969 attempts to fix #19930, however, broke the Toggle Selection functionality. This PR removes the for attribute which is not required since the checkbox is wrapped in the label tag and use the id for the Toggle Selection button.

Testing Instructions

Go to Components > Smart Search
Click Index button
Click Search Filters
Click New button
Go to a section and click the Toggle Selection button

Expected result

Toggle Selection button toggles the checkboxes.

Actual result

Toggle Selection button does not toggle the checkboxes.

Documentation Changes Required

none

avatar Quy Quy - open - 15 May 2018
avatar Quy Quy - change - 15 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 May 2018
Category Front End com_finder
avatar Quy Quy - change - 15 May 2018
Title
[com_finder] Fix toggle selection
[com_finder] Fix toggle selection (Regression ##19969)
avatar Quy Quy - edited - 15 May 2018
avatar Quy Quy - change - 15 May 2018
Title
[com_finder] Fix toggle selection (Regression ##19969)
[com_finder] Fix toggle selection (Regression #19969)
avatar Quy Quy - edited - 15 May 2018
avatar infograf768 infograf768 - test_item - 16 May 2018 - Tested successfully
avatar infograf768
infograf768 - comment - 16 May 2018

I have tested this item successfully on 107b943


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

avatar HLeithner HLeithner - test_item - 16 May 2018 - Tested successfully
avatar HLeithner
HLeithner - comment - 16 May 2018

I have tested this item successfully on 107b943


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 16 May 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 16 May 2018

Ready to Commit after two successful tests.

Thanks for Tests, guys

avatar mbabker mbabker - change - 18 May 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-05-18 11:12:36
Closed_By mbabker
Labels Added: ? ?
avatar mbabker mbabker - close - 18 May 2018
avatar mbabker mbabker - merge - 18 May 2018

Add a Comment

Login with GitHub to post a comment