-
Notifications
You must be signed in to change notification settings - Fork 4.5k
[Feature proposal] Normalize align/justify utilities #1647
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
Comments
Hey @sebastiandedeyne thanks for the thoughtful proposal! I totally see the rationale behind this idea but personally I would be more inclined to leave the existing ones unchanged, and only use the more verbose naming for the new ones to disambiguate them from the existing ones, so:
Two reasons I would prefer this approach:
I'd rather optimize for the common use case (which also has the benefit of not being a breaking change) and have people use longer class names for the less used properties personally. We made a similar decision when introducing the |
I totally respect the decision of optimizing for the common use case. Thanks for the detailed response! |
The
align-*
andjustify-*
related utilities currently use a short representation, e.g.items-center
foralign-items: center
. This makes sense forflex
, since you can get most things done withalign-items
andjustify-content
.With
grid
being part of Tailwind now, additionalalign-*
andjustify-*
properties became more prevalent. Some are missing, and some current utilities become ambiguous when filling the missing gaps. E.g. isjustify-items
utilities were to be added, it wouldn't be obvious anymore thatitems-center
relate toalign-items
. Then there's also theplace-*
properties which will probably be added at some point too.We could normalize the properties in a non-breaking way in 1.x by adding more utilities, and removing the old ones in 2.x.
align-items
items-*
align-items-*
align-content
content-*
align-content-*
align-self
self-*
align-self-*
justify-items
justify-items-*
justify-content
justify-*
justify-content-*
justify-self
justify-self-*
Related issues: #1628, #1354, and maybe #1517.
The text was updated successfully, but these errors were encountered: