Skip to content
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

Menu Colors in Skins with a default Dark/Light also #5886

Merged
merged 1 commit into from
Feb 13, 2022

Conversation

baconpaul
Copy link
Collaborator

New feature in skins menu - menus always dark/light/follow skin
Colors in skin which map to current dark menus by default
Light mode works and sticks

Closes #5861

New feature in skins menu - menus always dark/light/follow skin
Colors in skin which map to current dark menus by default
Light mode works and sticks

Closes surge-synthesizer#5861
@mkruselj
Copy link
Collaborator

I think I would've preferred per skin colors for the menus rather than a global option, but easy to modify down the road.

@baconpaul baconpaul merged commit e0065b4 into surge-synthesizer:main Feb 13, 2022
@baconpaul baconpaul deleted the menu-5861 branch February 13, 2022 15:29
@baconpaul
Copy link
Collaborator Author

I think I would've preferred per skin colors for the menus rather than a global option, but easy to modify down the road.

since that exact feature is in the code, i'm not sure i understand your comment.

@mkruselj
Copy link
Collaborator

Right I meant just not necessary to have a userdefaults thingie, instead hardcode them to each skin so that it always matches.

@baconpaul
Copy link
Collaborator Author

Yeah I thought about using the os light dark default also. Most apps I have used don’t override the menus with settings so I wanted an option to skip out of that and just have light. But the default is to use whatever is in the skin

@baconpaul
Copy link
Collaborator Author

If we had user skin color overrides we could of course get rid of the sticky option

@mkruselj
Copy link
Collaborator

Is there any way we can get OS light/dark setting then follow it, then just have a single menu option to toggle using menu skin colors from the skin?

@baconpaul
Copy link
Collaborator Author

I know how to do that on Mac but not on win yet but sure it’s just an api

@mkruselj
Copy link
Collaborator

Hmmm apparently not, we gotta read registry.

@baconpaul
Copy link
Collaborator Author

Ha it's funny. I just committed to sstplugin infra saying 'it will all work we just gotta read the registry'

the registry is the api. that's one of the worst things about windows, but it's also true!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Menu tweaks
2 participants