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

Rename max-width and max-height utilities from mw-*/mh-* to max-w-*/max-h-* #40674

Open
2 tasks done
alexandergitter opened this issue Jul 29, 2024 · 3 comments · May be fixed by #41236
Open
2 tasks done

Rename max-width and max-height utilities from mw-*/mh-* to max-w-*/max-h-* #40674

alexandergitter opened this issue Jul 29, 2024 · 3 comments · May be fixed by #41236
Labels

Comments

@alexandergitter
Copy link
Contributor

Prerequisites

Proposal

I suggest renaming the utilities for setting max-width and max-height. With the default values these are currently mw-100 for max-width: 100% and mh-100 for max-height: 100%.

These should be renamed to max-w-100 and max-h-100.

Motivation and context

mw is ambiguous when it comes to max-width vs min-width. A custom min-width utility in Sass cannot use the mw name since that's already taken for max-width.

(Same argument for mh.)

@plutocrate
Copy link

@alexandergitter I will like to work on this issue.

@alexandergitter
Copy link
Contributor Author

@alexandergitter I will like to work on this issue.

@fragtheboy Please note that I'm not a maintainer for this project, so I can't speak to whether this proposal or a pull request would be accepted at this point.

@plutocrate
Copy link

@alexandergitter sure, noted.

@mdo mdo added css v6 and removed feature labels Feb 27, 2025
@mdo mdo added this to v6.0.0 Feb 27, 2025
@github-project-automation github-project-automation bot moved this to Inbox in v6.0.0 Feb 27, 2025
@mdo mdo linked a pull request Feb 27, 2025 that will close this issue
56 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Inbox
Development

Successfully merging a pull request may close this issue.

3 participants