docs: update frontend contributing guide for shadcn/ui and TailwindCSS migration #18702
+343
−30
Conversation
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
…S migration Update the frontend contributing guide to reflect the current architecture and ongoing migration from MUI to shadcn/ui and from Emotion to TailwindCSS. Key changes: - Updated tech stack to include TailwindCSS, shadcn/ui, Lucide React, and Biome - Added migration status section with current progress (~210 MUI files, ~41 migrated) - Added development commands section with pnpm commands and pre-PR checklist - Rewrote UI components section to prioritize shadcn/ui over MUI - Completely updated styling section to focus on TailwindCSS - Enhanced accessibility section with TailwindCSS examples - Added comprehensive migration guide with practical examples and checklists - Documented semantic color system and best practices Co-authored-by: BrunoQuaresma <[email protected]>
Ran make fmt and frontend linting to ensure code quality: - Go formatting: passed - Terraform formatting: passed - Frontend formatting (Biome): passed (1124 files, no fixes needed) - Frontend linting (Biome): passed (1124 files, no issues) No changes were required as the code already meets all formatting standards. Co-authored-by: BrunoQuaresma <[email protected]>
Fixed 11 markdown linting errors found by markdownlint-cli2: - MD022: Added blank lines around headings - MD032: Added blank lines around lists (9 instances) - MD031: Added blank lines around fenced code blocks - MD040: Added language specification to fenced code block All markdown linting rules now pass with 0 errors. Co-authored-by: BrunoQuaresma <[email protected]>
There was a problem hiding this comment.
🚫 [linkspector] reported by reviewdog 🐶
Cannot reach https://stackoverflow.com/questions/69711888/react-testing-library-getbyrole-is-performing-extremely-slowly Status: 403
coder/docs/about/contributing/frontend.md
Line 502 in 2de7330
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Update the frontend contributing guide to reflect the current architecture and ongoing migration from MUI to shadcn/ui and from Emotion to TailwindCSS.
Key changes: