Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(Tailwind): Rename utilities used in our code #1044

Merged
merged 1 commit into from
Jan 27, 2025
Merged

Conversation

qgerome
Copy link
Member

@qgerome qgerome commented Jan 27, 2025

There was some missing updates in the code according to their upgrade guide.

@qgerome qgerome requested a review from YolanFery January 27, 2025 09:37
Copy link
Contributor

@YolanFery YolanFery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks ! How did you find those ? What approach did you use to find the missing cases ?

@qgerome
Copy link
Member Author

qgerome commented Jan 27, 2025

Thanks ! How did you find those ? What approach did you use to find the missing cases ?

I launched my browser and saw that some elements were quite unaesthetic. I then looked at the upgrade guide here and checked the "removed deprecated utilities & renames utilities"

@qgerome qgerome merged commit a3feb7b into main Jan 27, 2025
3 checks passed
@qgerome qgerome deleted the fix-tailwind-upgrade branch January 27, 2025 12:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants