feature: add Light Mode Only mode#33
Open
devin-ai-integration[bot] wants to merge 1 commit into
Open
Conversation
Co-Authored-By: shahmir.masood <shahmir.masood@cognition.ai>
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
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
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.
Summary
Adds a Light Mode Only setting that locks the app to the light theme and disables the Dark Mode toggle. Useful for users who never want the app to render in dark mode (e.g. shared/kiosk setups). The preference persists via
localStorageunderlight-mode-only.When enabled it forces the theme to light and disables the Dark Mode row so it can't be turned back on:
The
Switchcomponent gained adisabledprop (nativedisabled+ dimmed.disabledstyle + click guard), used to lock the Dark Mode row while Light Mode Only is active.Files
src/App.jsx: newlight-mode-onlylocalStorage state,handleLightModeOnly, dark-mode guard, props toSettingModal.src/components/SettingModal/SettingModal.jsx: new "Light Mode Only" row; Dark Mode row disabled when active.src/components/Switch/Switch.jsx+.module.scss:disabledprop support and styling.Verification
Ran
npm run check(Prettier) andnpm run build— both pass. Manually tested in the browser: enabling Light Mode Only switches the UI to light and locks the Dark Mode toggle (clicking it does nothing).Link to Devin session: https://app.devin.ai/sessions/09f6572e8db04aeeb65c2b28bafdf34d
Requested by: @shahmir-masood
Devin Review