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

feat: match the theme color of status bar in light/dark mode #3358

Closed
coxde opened this issue Jun 21, 2024 · 3 comments
Closed

feat: match the theme color of status bar in light/dark mode #3358

coxde opened this issue Jun 21, 2024 · 3 comments

Comments

@coxde
Copy link

coxde commented Jun 21, 2024

Is your feature request related to a problem? Please describe.
Now the PWA theme_color is hard-coded to #343541 (according to my search), which makes the status bar doesn't looks like a native app in light mode.

1

Describe the solution you'd like
I'm not sure but there should be a method to match the color.

Describe alternatives you've considered
N/A

Additional context
See also:
https://developer.mozilla.org/en-US/docs/Web/Progressive_web_apps/How_to/Customize_your_app_colors
https://developer.mozilla.org/en-US/docs/Web/CSS/@media/prefers-color-scheme
elk-zone/elk#941
cheeaun/phanpy#231

@tjbck
Copy link
Contributor

tjbck commented Jun 21, 2024

Just pushed a commit to dev, let me know if that did anything for you!

@coxde
Copy link
Author

coxde commented Jun 21, 2024

Thanks for your quick response! Now it can change color based on light/dark mode.

One nitpick is that the white/black color and the status bar don't exactly match the white/black of the app theme, it would be better if they could match!

@tjbck
Copy link
Contributor

tjbck commented Jun 22, 2024

The updated implementation delegates setting the colour to the device, further improvement won't be possible unless we overhaul the manifest.json code entirely. Thanks for the feedback!

@tjbck tjbck closed this as completed Jun 22, 2024
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

No branches or pull requests

2 participants