Skip to content
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

Add size-text shorthand #12292

Closed
wants to merge 1 commit into from

Conversation

giuseppeg
Copy link

This PR adds a size-text utility that sets width and height to 1em.

This is super useful when making icons and other elements that can then be controlled by setting a font-size on the parent or the element itself.

Relates to #12287

@adamwathan
Copy link
Member

Hey thanks for the PR! For now going to say no to this because we don’t include this size anywhere else in the framework, so it’s a bit of a bigger decision and can’t prioritize thinking it through right now. Maybe in the future though 👍🏻

@adamwathan adamwathan closed this Oct 25, 2023
@giuseppeg
Copy link
Author

No worries Adam! I submitted this because I use it on every project and have seen others doing the same too but I am sure you have a better overview of what is widely needed and what is not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants