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

grid-cols arbitrary value not working as expected #12281

Closed
squelix opened this issue Oct 25, 2023 · 3 comments · Fixed by #12283
Closed

grid-cols arbitrary value not working as expected #12281

squelix opened this issue Oct 25, 2023 · 3 comments · Fixed by #12283
Assignees

Comments

@squelix
Copy link

squelix commented Oct 25, 2023

What version of Tailwind CSS are you using?
v3.3.4

What build tool (or framework if it abstracts the build tool) are you using?
Next.js 13.5.6

What version of Node.js are you using?
v16

For example: v16

What browser are you using?
Chrome

What operating system are you using?
macOS

Describe your issue

I have this arbitrary value in className

"grid grid-cols-[repeat(3,_minmax(0,_max-content))]"

It's not generating the good css.

grid-template-columns: repeat(3,minmax(0,max - content));

There is spaces around the dash.

In tailwind v3.3.3 it was working, so I downgrade the version.

@DarkPromise
Copy link

Having the same issue here when using "grid-cols-[repeat(5,minmax(auto,max-content))]"
The new spaces injected causes the CSS to break and become invalid.

@RobinMalfait RobinMalfait self-assigned this Oct 25, 2023
@Sky-Flare
Copy link

Sky-Flare commented Oct 25, 2023

I have the same issue with this line
h-[calc(100%-env(safe-area-inset-bottom,0)-var(--main-size)-var(--bottom-size))]
which is transformed into:
height: calc(100% - env(safe - area - inset - bottom,0) - var(--main-size) - var(--bottom-size));

@RobinMalfait
Copy link
Member

Hey, thank you for this bug report! 🙏

We introduced a regression here, but this should be fixed by #12283, and is available in the latest release (3.3.5). Sorry for the inconvenience!

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 a pull request may close this issue.

4 participants