Rwq3/style migration#687
Open
YottaYocta wants to merge 26 commits into
Open
Conversation
This commit completes the migration from CSS modules to Tailwind CSS utility classes across the entire frontend codebase. Migration Summary: - Removed all 51 CSS module files (.module.css) - Migrated 71+ component files to use Tailwind CSS classes - 122 files changed, 714 insertions, 6667 deletions - All component functionality and visual appearance preserved Key Changes: - Removed all CSS module imports from TypeScript/TSX files - Converted inline style objects to Tailwind utility classes - Preserved dynamic styling patterns using classnames/cn utility - Maintained responsive breakpoints and hover/focus/active states - Refactored complex components (tables, modals, forms, etc.) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This reverts commit 6356258.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for carriage-web ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Member
|
[diff-counting] Significant lines: 2255. This diff might be too big! Developer leads are invited to review the code. |
anika-4444
requested changes
Apr 16, 2026
anika-4444
left a comment
anika-4444
left a comment
Contributor
There was a problem hiding this comment.
Overall looks pretty good! I just noticed some small styling problems that got translated weirdly in tailwind.
Hopefully I got all of them but maybe another look after you implement the changes will reveal the less obvious ones
| <h1 className={styles.heading}>Login</h1> | ||
| <div className="rounded-r-2xl bg-white grow flex flex-col justify-between items-center text-center p-16 relative max-md:rounded-b-2xl max-md:rounded-tr-none"> | ||
| <div className="flex flex-col justify-center items-center flex-1"> | ||
| <h1 className="text-3xl">Login</h1> |
Contributor
There was a problem hiding this comment.
seems like the styling here got a little lost
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
more styl