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

Fixed definition of place-content-* #15440

Merged
merged 5 commits into from
Dec 19, 2024
Merged

Fixed definition of place-content-* #15440

merged 5 commits into from
Dec 19, 2024

Conversation

thecrypticace
Copy link
Contributor

The utilities place-content-between, place-content-around, and place-content-evenly are defined incorrectly. This PR fixes them.

@thecrypticace thecrypticace requested a review from a team as a code owner December 18, 2024 23:05
@adamwathan adamwathan merged commit ab49b59 into next Dec 19, 2024
5 checks passed
@adamwathan adamwathan deleted the fix/v4-place-content branch December 19, 2024 01:17
@jspm2013
Copy link

jspm2013 commented Jan 25, 2025

sorry guys but it seems (at least for me, just installed v4.0.0) that the compiled css value is still "between" instead of "space-between", applying "place-content-between" on elements

EDIT / SOLVED: don't only update tailwind, but also tailwind/postcss ✌️

@thecrypticace
Copy link
Contributor Author

You sure you're not on an earlier version?

It looks right to me:

Screenshot 2025-01-25 at 13 27 32

@jspm2013
Copy link

aaargh...okay thank you @thecrypticace - even though I deleted and reinstalled node_modules and stuff before installing 4.0.0 it didn't work out, but yeah...user fault as usual - I forgot to also update tailwind/postcss, which was still on beta8... thank you for the great work!

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.

3 participants