This is a child of #3324398: [META] Update Claro CSS with new coding standards and part of #3254529: [PLAN] Drupal CSS Modernization Initiative.
The stylesheet at https://git.drupalcode.org/project/drupal/-/blob/10.0.x/core/themes/clar... needs to be refactored to make use of modern CSS and Drupal core's PostCSS tooling.
@todo: Add clear testing instructions to test this manually on the UI.
variables.pcss.css) where appropriate. Follow the proposed Drupal CSS coding standards to name the variables.font-size: 1.23rem; /* @todo One off value. */.entity-meta {
--entity-meta-title-font-size: var(--font-size-h5);
... more style
}
.entity-meta__title {
font-size: var(--entity-meta-title-font-size);
}Out of scope
None. There should be no visual differences.
Please post before/after screenshots and make sure they look the same.
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | 3332462-14-after.png | 51.92 KB | starshaped |
| #14 | 3332462-14-before.png | 50.95 KB | starshaped |
| #6 | Screenshot 2023-02-22 at 11.44.48 AM.png | 127.65 KB | gauravvvv |
| #6 | Screenshot 2023-02-22 at 11.51.15 AM.png | 128.55 KB | gauravvvv |
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes, plain diff MR !3509
Comments
Comment #2
starshapedActively working on this at Florida Drupal Camp 2023.
Comment #3
starshapedComment #4
gauravvvv commentedComment #6
gauravvvv commentedHere are the before/after patch screenshots. please review
Before patch:
After patch:
Comment #7
smustgrave commentedThanks for the screenshots
Appears there are failures in the MR.
Comment #8
starshapedI'm going to be working on this tonight and tomorrow to further clean up the CSS and fix the MR failures.
Comment #9
smustgrave commentedSounds good! But needs review isn’t the right status if there’s work to still be done.
Comment #10
starshapedYeah, that was a mistake on my end, I didn't realize I had changed the status! I'll set it to needs review once I get my work done.
Comment #11
smustgrave commentedWill keep an eye out to re-review
Comment #12
starshapedCode has been cleaned up and tests are passing. Ready for review!
Comment #13
smustgrave commentedSince you made additional changes since the last set of screenshots. Think we are going to need new ones since the .css file has changed.
Comment #14
starshapedReadded the RTL styles as they only apply on an RTL layout, and added before and after screenshots.
Comment #15
smustgrave commentedFrom what I can see everything has been covered with the nesting, variables, etc.
Comment #18
lauriiiCommitted 138349b and pushed to 10.1.x. Thanks!
Discovered some pre-existing inconsistencies with the RTL styles. Opened a follow-up to address those #3344765: Inconsistencies in system-status-counter RTL styles.