-
-
Notifications
You must be signed in to change notification settings - Fork 778
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
[Feature Proposal] Allow 'grid-' part to be optional when specifying grid columns #3533
Closed
4 tasks done
Labels
Comments
AndrewBogdanovTSS
changed the title
[Feature Proposal] Allow grid part to be optional when specifying grid columns
[Feature Proposal] Allow 'grid-' part to be optional when specifying grid columns
Jan 26, 2024
I am ok with it. PR welcome :) |
AndrewBogdanovTSS
pushed a commit
to AndrewBogdanovTSS/unocss
that referenced
this issue
Jan 29, 2024
AndrewBogdanovTSS
pushed a commit
to AndrewBogdanovTSS/unocss
that referenced
this issue
Jan 29, 2024
AndrewBogdanovTSS
pushed a commit
to AndrewBogdanovTSS/unocss
that referenced
this issue
Jan 29, 2024
AndrewBogdanovTSS
added a commit
to AndrewBogdanovTSS/unocss
that referenced
this issue
Jan 29, 2024
@antfu ready for review |
github-merge-queue bot
pushed a commit
that referenced
this issue
Mar 14, 2024
… (#3542) Co-authored-by: Chris <hizyyv@gmail.com> Co-authored-by: Anthony Fu <anthonyfu117@hotmail.com>
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Clear and concise description of the problem
As a developer using UnoCSS I want to be able to use shorter utilities when spcifying grid column sizes
Suggested solution
https://github.com/search?type=code&q=repo:unocss/unocss%20%22%2F%5Egrid-(rows%7Ccols)-(%5Cd%2B)%24%2F%22
grid-
in the above regular expression should be optional pretty much the same as it's already done for grid auto flow utilityunocss/packages/preset-mini/src/_rules/grid.ts
Line 54 in 8505f48
Alternative
Right now I just have to manually keep those rules in the project
but it would be great if that would come out of the box, given that the change is just about adding
?:
to the regexpAdditional context
No response
Validations
README.md
of using the package.The text was updated successfully, but these errors were encountered: