User tests: Successful: Unsuccessful:
Pull Request for Issue #27555
As suggested by @C-Lodder add additional click eventlistener in showon.js
Apply path, and run npm i to rebuild JavaScript files with this PR
Add config form field type radio with class "switcher btn-group btn-group-yesno"
Add an additional field that has a shownon dependency on the radio fiels
clicking yes should show the additional field, no should hide the additional fields
without this change, the showon functionality would not work on this radio field. Omitting the "btn-group btn-groupyesno" would fix that but then the same config file would break Joomla 3.x showon behavior.
None
@C-Lodder for coming up with the solution and everybody in advance for testing this PR
Status | New | ⇒ | Pending |
Category | ⇒ | JavaScript Repository NPM Change |
Hint for testers: It needs to run npm to compile the js. @Ruud68 could you add a corresponding hint to your testing instructions? Testers would find it more easily there than here.
Take it this now doesn't work anymore with the patchtester? Anyway, added a hint. my guess is that everybody who responded in the issue knows what to to and can test or has his own test case.
I have tested this item
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
?
?
NPM Resource Changed
|
Thanks all for testing!
I could if I was not one of the testers.
He's busy playing poker with me
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-06-06 18:21:52 |
Closed_By | ⇒ | wilsonge | |
Labels |
Thanks!
I tried the solution given above but a new issue found that in J4 if i use class "switcher btn-group btn-group-yesno" there are two label displayed.
Here is my Field:
Disable
Enable
i think the field name displayed two times, as i tried removing label and description from field too.
is there any Solution or fix for that?
Thanks
Hint for testers: It needs to run npm to compile the js. @Ruud68 could you add a corresponding hint to your testing instructions? Testers would find it more easily there than here.