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

android: fix navigation bar always being bright even in dark mode #3464

Merged
merged 2 commits into from
Apr 12, 2024

Conversation

kisaragi-hiu
Copy link
Contributor

Right now both light mode and dark mode get light navigation bar, and looks jarring in the dark modes.

This PR applies a more appropriate color before the UI thread runs (in app config), as well as applying the current theme background after the UI shell is mounted.

This should fix #3332.

Before After
Before Splash.tsx takes over Screenshot_20240410-160920 Screenshot_20240410-160105
After Splash.tsx Screenshot_20240410-160929 Screenshot_20240410-160010
Dark Screenshot_20240410-161036 Screenshot_20240410-153333
Dim Screenshot_20240410-161032 Screenshot_20240410-153320
Light Screenshot_20240410-161039 Screenshot_20240410-153325

@kisaragi-hiu kisaragi-hiu changed the title android: apply theme color on navigation bar android: fix navigation bar always being bright even in dark mode Apr 12, 2024
Right now both light mode and dark mode get light navigation bar, and
looks jarring in the dark modes.

This commit applies a more appropriate color before the UI thread
runs (in app config), as well as applying the current theme background
after the UI shell is mounted.

This should fix bluesky-social#3332.
@kisaragi-hiu kisaragi-hiu force-pushed the android-navbar-color branch from e3e6d74 to 7c60395 Compare April 12, 2024 20:54
Copy link
Contributor

@haileyok haileyok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonderful, thank you for doing this!

@haileyok haileyok merged commit cefa0a5 into bluesky-social:main Apr 12, 2024
6 checks passed
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.

Android bottom navigation bar is always in light colors
2 participants