app theme for zplit#51
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
fix this typo app_text_Styles to app_text_styles |
|
rest LGTM |
M4dhav
left a comment
There was a problem hiding this comment.
Use ThemeData class to create a theme instead of defining colors like this.
M4dhav
left a comment
There was a problem hiding this comment.
We don't need app_colors.dart or app_text_styles.dart as app_theme is importing these variables. Just define the colors and Styles in the Theme directly.
|
@M4dhav what you think now ? |
Addressed Issues:
Fixes #50
Screenshots/Recordings:
No UI changes — this PR adds theme configuration files only.
Additional Notes:
Added app color palette and text styles for the Zplit app theme, including light and dark mode colors, typography
AI Usage Disclosure:
We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact. AI slop is strongly discouraged and may lead to banning and blocking. Do not spam our repos with AI slop.
Check one of the checkboxes below:
I have used the following AI models and tools: TODO
Checklist