-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add UI settings #1055
base: master
Are you sure you want to change the base?
Add UI settings #1055
Conversation
JosteinLindhom
commented
Feb 28, 2024
- feat(state): add settings namespace to Overmind configuration
- feat(ui): set up route and button for a settings page
- feat(ui): add settings components for changing UI elements
- feat(ui): add theme provider that update CSS variables when state changes
- fix: add missing settings route
The settings namespace is used to keep track of and persist (to local storage) user selected settings.
Current implemented setting options include: - Selecting between three predefined color pairs for the pass/fail indicator for tests / criterion - Selecting custom colors for the above (this might be overkill) - Adjusting the width of the pass/fail indicator
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1055 +/- ##
==========================================
- Coverage 40.39% 40.23% -0.17%
==========================================
Files 110 91 -19
Lines 9019 6935 -2084
==========================================
- Hits 3643 2790 -853
+ Misses 5041 3845 -1196
+ Partials 335 300 -35
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Should this be linked with #1049 and taken out of draft mode? |