Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Issue #2473949 by mortendk, rteijeiro, rachel_norfolk, Cottser, LewisNyman: Prefix form-type-* classes with js-
Task
Prefix form-type-* classes with js- to separate classes needed for JavaScript functionality from those used for styling and make it clear which classes can safely be removed without breaking JavaScript functionality.
Remaining tasks
- Patch
- Patch review
- Manual testing
Steps to test
- Navigate to
admin/config/people/accounts
- Confirm that the vertical tabs are functional
Beta phase evaluation
Issue category | Task |
---|---|
Issue priority | Normal |
Unfrozen changes | Unfrozen because it mostly just affects templates and JS which are not frozen. |
Prioritized changes | The main goal of this issue is to improve themer experience and arguably it reduces fragility where JavaScript and markup intersect. |
Disruption | Shouldn't be too disruptive as it is mostly affecting CSS classes that are added to markup. Themes extending Classy will only have classes added. Themes not extending Classy will have classes removed but they can be added back via template overrides. |
User interface changes
None for themes extending Classy. Possible visual changes for other themes.
API changes
n/a
Comment | File | Size | Author |
---|---|---|---|
#5 | 2473949-5.patch | 4.25 KB | star-szr |
Comments
Comment #1
star-szrInitial patch split from the parent, some additional changes from grepping, and interdiff between the two.
Comment #3
LewisNymanol>
This class is used for styling in Bartik and I can't find where it is used in javascript. We should not convert it to a js class
I think we should only test for the functional classes
Comment #4
star-szrThanks for the review. tour.module is where the class is used for JavaScript!
And to be clear:
As for point 2, see #2474107: Make OptionsWidgetsTest::testEmptyValue() care less about markup :)
Comment #5
star-szrThis just rolls back the changes to OptionsWidgetsTest now that #2474107: Make OptionsWidgetsTest::testEmptyValue() care less about markup is in. Also this needs commit credits added for the folks that worked on the original, now meta issue.
Comment #6
star-szrAdding suggested commit message.
Comment #7
LewisNymanAhhh ok, that wasn't setting the class, that was using the class. RTBC!
Comment #8
LewisNymanAhem...
Comment #9
alexpottCommitted b48bfcb and pushed to 8.0.x. Thanks!