-
Notifications
You must be signed in to change notification settings - Fork 655
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Organize CSS files following 7-1 pattern style #14843
Conversation
@import 'utilities/colors'; | ||
@import 'utilities/flexbox'; | ||
@import 'utilities/spacing'; | ||
@import 'utilities/utilities'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe, instead of utilities
, in order to avoid repeating the word (utilities/utilities
) we could use helpers
, general
, common
, or something similar. What do you think? Maybe it's a pattern and I'm missing something there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've seen abstracts, helpers and utilities. What I like about utilities is that it starts with letter U so main.scss
can almost be ordered alphabetically and utilities will be at the end of the file 😄But it's true that the repeating the word (utilities/utilities) is a bit awkward. So changing it to helpers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I left a comment but it's a suggestion, not a request change 👍 🚀
I have moved the
.scss
files in a similar way to 7-1 pattern. It is not exactly the same. I wanted to keepvariables.scss
in the root because they are used in all the components. Also I have renamed some files and moved some code to where I think it fits best.https://sass-guidelin.es/#the-7-1-pattern
Acceptance
Check that the New Dashboard is displayed correctly.
Related Issue
#14670