Skip to content

chore: replace tailwind with css modules #278

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

Merged
merged 11 commits into from
May 16, 2025
Merged

chore: replace tailwind with css modules #278

merged 11 commits into from
May 16, 2025

Conversation

qw-in
Copy link
Contributor

@qw-in qw-in commented May 16, 2025

Fixes #252

My goal here was visual parity, not good css. Duplication and odd class names to be expected. This will just be a starting point and I expect to replace or clean up most of these styles over the coming days (or migrate to our design system).

Steps taken, after each step playwright was used to sanity check:

  1. Port all tailwind classes into css modules 1 to 1 via @ apply (thanks ai)
  2. Expand all module files with the tailwind cli, and finally expand the global.css
  3. Remove the dependency

I'm sure I missed something along the way, I'll be doing more visual comparisons before I merge

@qw-in qw-in self-assigned this May 16, 2025
@qw-in qw-in requested a review from a team as a code owner May 16, 2025 02:40
Copy link

vercel bot commented May 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
arcjet-js-example ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 16, 2025 2:40am

@qw-in qw-in requested review from a team and morekid May 16, 2025 02:47
@qw-in
Copy link
Contributor Author

qw-in commented May 16, 2025

Oh, I should update the readme :)

Copy link
Contributor

@davidmytton davidmytton left a comment

Choose a reason for hiding this comment

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

Nice one. I love seeing dependencies removed 😎

@qw-in qw-in removed the request for review from morekid May 16, 2025 14:03
@qw-in qw-in merged commit cbfa851 into main May 16, 2025
5 checks passed
@qw-in qw-in deleted the quinn/remove-tailwind branch May 16, 2025 14:47
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.

Replace tailwindcss and shadcn
2 participants