-
Notifications
You must be signed in to change notification settings - Fork 373
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 app-wide themes #2258
Add app-wide themes #2258
Conversation
setTheme(R.style.AppTheme_Unified); | ||
} | ||
|
||
if (_appSettings.getAppAccentName().contains("black_white")) { |
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.
When it's done this way, next time people will complain that they can't customize colors. While it's more possibilities, it's still limited to a small amount of hardcoded colors this way.
@@ -13,7 +13,7 @@ | |||
|
|||
name: "CI" | |||
|
|||
on: [push, pull_request] | |||
on: [push, pull_request, workflow_dispatch, workflow_call] |
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.
please revert changes to the workflow files. You can of course keep whatever personal you do....in your own branches.
ignore my commit for workflow part. I was trying to get my workflow to work and publish apk to my repo.
This is start of theme system / chooser for markor.
I know I need to add more themes to match to the ones in the notepad part.
I think people should be able to change accents / icons colors and other colors esp on the main app.