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

Rename spin animation #1333

Merged
merged 1 commit into from May 29, 2021
Merged

Conversation

mdix
Copy link
Contributor

@mdix mdix commented May 27, 2021

The spin animation defined by selectr collides with the spin
animation defined by tailwind. It overrides the animation and causes
the spinning elements to spin offcenter. By renaming it we kinda
namespace it and prevent a collision.

Definition of tailwind spin animation

The spin animation defined by selectr collides with the spin
animation defined by tailwind. It overrides the animation and causes
the spinning elements to spin offcenter. By renaming it we kinda
namespace it and prevent a collision.
@josdejong
Copy link
Owner

Thanks, makes sense.

Maybe a good idea if tailwind would also use prefixes for their class names 😁 . I remember Bootstrap also giving these kind of conflicts due to too generic class names.

@josdejong josdejong merged commit 0fabe6f into josdejong:develop May 29, 2021
@mdix
Copy link
Contributor Author

mdix commented May 29, 2021

Hey! Thanks for merging. Actually, they did for v2 (tailwindlabs/tailwindcss#2621), but v1.9.6 is unprefixed and if you need IE11 support you can't update.

@josdejong
Copy link
Owner

Ahh that's good to hear!

@josdejong
Copy link
Owner

Fixed now in v9.4.2

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.

None yet

2 participants