-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Fix some MAP issues #373
Fix some MAP issues #373
Conversation
Merge TheIOFoundation/ProjectLockdown "master" branch
Merge TheIOFoundation/ProjectLockdown "master" branch
Merge TheIOFoundation/ProjectLockdown "master" branch
Fix UI bugs (#222)
Deploy preview for dazzling-visvesvaraya-f47271 ready! Built with commit 8eb6428 https://deploy-preview-373--dazzling-visvesvaraya-f47271.netlify.app |
t: s => { | ||
switch (s) { | ||
case 'menu.userPreferenceSection.theme.action': | ||
return 'Toggle '; | ||
return 'Switch to '; | ||
case 'menu.userPreferenceSection.theme.light': | ||
return 'Light mode'; | ||
case 'menu.userPreferenceSection.theme.dark': | ||
return 'Dark mode'; | ||
case 'menu.userPreferenceSection.theme.dark': | ||
return 'Light mode'; | ||
default: | ||
return s; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this block obsolete? can you try removing it to confirm it all still works?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
locale
itself needs to be passed to <Settings>
, but it still works without the function.
Kudos, SonarCloud Quality Gate passed! |
Resolves #365, resolves #367, and resolves #369. Fixes most of #368 except for the final point.