Improvement/interface 2 #127
Merged
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
…namic color modes
…to use the lib folder
There was a problem hiding this comment.
Pull Request Overview
This PR implements theme support with light/dark mode toggle and refactors the codebase structure by moving constants to a centralized location. Key changes include adding color mode support across components, reorganizing validation and category constants, and improving UI consistency.
- Added light/dark theme toggle with Chakra UI's color mode system
- Refactored constants from
/constto/lib/constwith new helper functions - Updated multiple components to use dynamic colors based on theme mode
Reviewed Changes
Copilot reviewed 18 out of 20 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| src/styles/theme.ts | Configured theme with dark mode as default and dynamic color styles |
| src/lib/const/categories.ts | Added helper functions and types for category management |
| src/lib/const/validation.ts | Moved validation constants to centralized location |
| src/pages/auth/signup.tsx | Updated imports and replaced execute() with execute?.() for recaptcha |
| src/components/Header/Profile.tsx | Added theme toggle button in user menu |
| src/features/user/profile/index.tsx | Applied dynamic theme colors and removed unused icon import |
| src/pages/spots/index.tsx | Updated to use hardcoded dark mode colors instead of dynamic colors |
| src/features/user/edit/index.tsx | Applied consistent heading style but uses hardcoded dark colors |
| src/components/StoriesSwiper/index.tsx | Replaced placeholder images and removed unused button functionality |
| src/components/CardUser/index.tsx | Improved card styling with theme-aware colors and consolidated social media functions |
| src/components/Sidebar/SidebarNav.tsx | Moved "Cadastro geral" link to authenticated user section with dynamic colors |
| src/features/skatistas/index.tsx | Applied theme-aware background colors |
| src/features/stories/home/index.tsx | Applied dynamic title background color |
| src/features/login/modal/login.tsx | Added theme-aware colors for buttons and backgrounds |
| src/components/Footer/index.tsx | Applied theme-aware colors to footer elements |
| src/types/usersBasics.type.ts | Added id field to category object |
| src/types/UserBasicsWithPagination.type.ts | Added id field to category object |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…t color for improved styling
…n Header component
… in Header component
Owner
Author
|
This pull request has been deployed to Vercel.
|
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.
No description provided.