Skip to content

Conversation

@egeakman
Copy link
Member

The hidden class no longer seems to work along with other classes that affect display.

I had to use a workaround like this:

${className.includes("hidden") ? "hidden" : "inline-block"}

Also, there seems to be a new preferred way of configuration: inline configuration in CSS. I like it.

I had to tweak some components and pages to make this work, but I most likely missed something somewhere.

Closes #1038

@egeakman egeakman added the dependencies Pull requests that update a dependency file label Mar 11, 2025
@egeakman egeakman requested review from artcz, cybit and hypha March 11, 2025 05:54
@egeakman egeakman self-assigned this Mar 11, 2025
@europython-society
Copy link
Member

europython-society commented Mar 11, 2025

Preview available

Key Value
url https://migrate-tailwind.ep-preview.click
last update 2025-03-11T05:55:30.427Z

@egeakman
Copy link
Member Author

@hypha This will likely create conflicts with the sponsor PR because the colors were defined in the tailwind config. How would you like to move forward with this 🤗

@egeakman
Copy link
Member Author

Let's hold this until the sponsor PR gets merged!

@egeakman
Copy link
Member Author

It is probably the safest option to hold this until after the conference, since it might affect existing and new pages.

@egeakman egeakman added the DO-NOT-MERGE Not ready to be merged yet label Mar 25, 2025
@egeakman egeakman closed this Mar 25, 2025
@egeakman egeakman deleted the migrate-tailwind branch April 13, 2025 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file DO-NOT-MERGE Not ready to be merged yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate tailwindcss to v4, and use vite it through vite plugin

3 participants