Refactor CSS variable names for consistency#6128
Refactor CSS variable names for consistency#6128kyrapieterse wants to merge 47 commits intomasterfrom
Conversation
Replaces long-form CSS custom property names (e.g., --maxi-primary-color) with shorter, standardized abbreviations (e.g., --mpc) across admin, starter sites, and library styles. Updates all relevant SCSS, CSS, and React icon usage to reference the new variable names, improving maintainability and consistency throughout the codebase.
|
Important Review skippedToo many files! 130 files out of 280 files are above the max files limit of 150. You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| </defs> | ||
| e xlinkHref="#scale__a" /> inkHref="#scale__b" transform="translate(.04 -.023) scale(.99998)" /> ref="#scale__c" transform="translate(0 .009) scale(.99998)" /> |
There was a problem hiding this comment.
The scale.svg markup is corrupted: instead of <use> elements referencing scale__a/b/c, line 7 now contains the literal text e xlinkHref... with no valid tags. This makes the “scale” setting icon render nothing wherever the raw SVG is used (e.g., in the settings panel or icon picker). The previous version had three <use> elements here; they need to be reinstated so the icon draws correctly.
Useful? React with 👍 / 👎.
|
Notes for @elzadj I was unable to rename maxi-blocks--active, maxi-container-block maybe there is a way you can. extremely heavy these 2 skipped items like maxi-hover-effect, maxi-background-displayer, maxi-slider-block, and maxi-image-block. Reason: These classes are saved in the database (static HTML). Renaming them now would break the styling of all existing content on your site. |
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
1 participant
An attempt at fixing the speed