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

Use primaryColor for status bar color #609

Merged
merged 1 commit into from
Jun 2, 2024
Merged

Conversation

nortio
Copy link
Contributor

@nortio nortio commented May 27, 2024

Fixes in my opinion an annoying visual issue:

statusbar-old

The status bar color does not match the app bar color. While I believe this is not intentional, in the case that it is it should at least be corrected to a color with better contrast with the status bar text.

This is now the result after the fix (also works in light mode):

Screenshot from 2024-05-27 19-40-38

There seems to be the same problem in the settings page but I did not fix it.

@Tobi823
Copy link
Owner

Tobi823 commented Jun 2, 2024

@all-contributors please add @nortio for code

@Tobi823
Copy link
Owner

Tobi823 commented Jun 2, 2024

Thanks

Copy link
Contributor

@Tobi823

I've put up a pull request to add @nortio! 🎉

@Tobi823 Tobi823 merged commit d157aef into Tobi823:master Jun 2, 2024
Tobi823 added a commit that referenced this pull request Jun 30, 2024
- Thanks nortio for fixing primaryColor for status bar color (#609)
- Thanks Andrew Gunnerson (chenxiaolong) for setting "update ownership" when installing a new app (#591)
- Thanks for the Weblate contributions from: Priit Jõerüüt, Cirnos, Coool (github.com/Coool), Retrial, Flavio F. M., gallegonovato, Scrambled777, 大王叫我来巡山, Kazushi Hayama, Giovanni Donisi, Kirill Isakov, 109247019824
- Dont check for updates / don't download updates when power save mode is enabled
- Dont use OkHttp to cache network responses, use custom data structure
- Try to fix download timeouts by increasing OkHttp timeout thresholds
- Try to fix bug by initialize WorkManager if it is not initialized (#586)
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.

2 participants