Skip to content

[4.0] Updated switcher.php layout and its corresponding scss files#32377

Merged
infograf768 merged 3 commits intojoomla:4.0-devfrom
thednp:patch-5
Feb 11, 2021
Merged

[4.0] Updated switcher.php layout and its corresponding scss files#32377
infograf768 merged 3 commits intojoomla:4.0-devfrom
thednp:patch-5

Conversation

@thednp
Copy link
Contributor

@thednp thednp commented Feb 10, 2021

Summary of Changes

Since the <legend> added to the switch field type is set to use visually-hidden class, with the self explanatory function, all the additional styling is redundant, hence we clean it up.

Pull Request for Issue #32367 (comment) .

Testing Instructions

Simplest way to test:

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

No visual or functional change is expected since the target element uses the .visually-hidden class

Documentation Changes Required

No

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Feb 10, 2021
@thednp thednp changed the title Update _form.scss [4.0] Update _form.scss Feb 10, 2021
@brianteeman
Copy link
Contributor

If you are removing the classes from the template css then you should also remove the reference to them in the php markup.

For reference the class was not invented for fun as it did serve a purpose when the code was originally committed by me at #24463.

The code was made useless by changes in #27684 which was when it should have been removed

@thednp
Copy link
Contributor Author

thednp commented Feb 10, 2021

@brianteeman roger, also updated. Thanks man.

@thednp thednp changed the title [4.0] Update _form.scss [4.0] Updated switcher.php layout and its corresponding _form.scss Feb 10, 2021
@Quy
Copy link
Contributor

Quy commented Feb 10, 2021

@thednp thednp changed the title [4.0] Updated switcher.php layout and its corresponding _form.scss [4.0] Updated switcher.php layout and its corresponding scss files Feb 10, 2021
@Quy
Copy link
Contributor

Quy commented Feb 10, 2021

I have tested this item ✅ successfully on d573b80


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

1 similar comment
@infograf768
Copy link
Member

I have tested this item ✅ successfully on d573b80


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

@infograf768
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 11, 2021
@infograf768 infograf768 added this to the Joomla 4.0 milestone Feb 11, 2021
@infograf768 infograf768 merged commit 9a57d95 into joomla:4.0-dev Feb 11, 2021
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Feb 11, 2021
@infograf768
Copy link
Member

Tks

@thednp thednp deleted the patch-5 branch February 11, 2021 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants