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

Drop default plugins "form", "typography", and "container-queries" #446

Merged
merged 1 commit into from
Dec 29, 2024

Conversation

flavorjones
Copy link
Member

@flavorjones flavorjones commented Dec 19, 2024

Typography and container queries aren't used by this gem directly.

The form plugin is used, but I've attempted to complete styling of the generator template forms without it (though there may still be some differences due to the absence of style reset).

See #443 and #441 for examples of users getting poor rendering due to conflicts between Tailwind UI and the form plugin.

@dhh Really it's up to you whether this kind of minimalism is in line with your vision for the gem? It does seem like some developers and designers want to start from a vanilla initialization that matches what other tailwindcss tools do (Vite, CDN).

Some alternative approaches are also discussed in the comments in #443.

taking care to fix up the form styling.
@dhh
Copy link
Member

dhh commented Dec 29, 2024

Happy to see these go. As long as everything from Tailwind UI remains copy/paste. I think that was probably the original intention behind adding these default plugins. That they were/thought to be required for Tailwind UI. So if they're getting in the way of that, good to dump.

@flavorjones flavorjones merged commit ee99b73 into main Dec 29, 2024
17 checks passed
@flavorjones flavorjones deleted the flavorjones-drop-default-plugins branch December 29, 2024 18:51
@flavorjones
Copy link
Member Author

Released in v3.1.0: Release v3.1.0 / 2024-12-29 · rails/tailwindcss-rails

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.

2 participants