-
Notifications
You must be signed in to change notification settings - Fork 0
New layout utilities #297
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
New layout utilities #297
Conversation
theoluciano
left a comment
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.
Do you want to repeat the description for each of the blocks when you're just changing one keyword (stack, split, cluster) at the end? Seems like
While flex utilities are great for moving quickly or for simple layouts, they can easily become cumbersome to manage and tend to turn into "Flex Spaghetti". In some of these cases, reaching for named components may make sense. However, there are times when a full component is more restrictive than you need and hurts productivity. Sometimes you need just a bit more structure to your layout, but not a component. could just be more of a top level thing and not need to be repeated. Just a thought.
Looks good otherwise; sheep it!
|
@theoluciano It could be a top level thing, I'm just not sure where it would belong. |
|
@theoluciano I moved that code block to a full page for utilities and pulled in the writing Scott and I did a while back but never added to Optics.
|
dallasbpeters
left a comment
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 ![]()
|
@Jeremy-Walton With these new layout utilities, can we start prefixing the classes with op- to future-proof these a bit? I’m thinking that since these are new to the system, we can remove some breakage in the future as we introduce class prefixing more broadly into the design system. |
:where(.stack) {
gap: var(...);
} |
… be overridden with other utilities




Why?
Add some new flex utilities for more declarative layouts without going the full stretch to make components.
What Changed
Quality Assurance
[ ] Have you updated any usage of changed tokens?[ ] Did you add a component?[ ] Have you added it to the dependency graph?[ ] Have you added it to the docs?[ ] Did you update a component?[ ] Have you updated the dependency graph?[ ] Have you updated the docs?[ ] Do you need to update the package version?Planning on one more addition before release
v2.3.0Screenshots
Cluster
Split
Stack
Utility Layout