-
-
Notifications
You must be signed in to change notification settings - Fork 780
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
In selected state, illegible title due to low contrast to background #527
Comments
@mediter There is no need to comment that the issue is still occurring. We'll close this issue when it gets fixes. |
@insidegui I followed the trail here for adhering to system theming, apologies for commenting on an old issue with an unrelated topic, but I felt it appropriate to ask before opening a new issue. Out of pure curiosity, why the persistent dark mode over adhering to the system? |
It’s a media consumption app, it was a design choice. |
I'll play around with it and see how it looks. Might be able to get it looking nice in both themes. If not, oh well. I have a thing for the light/dark theming (don't lean either way, but am a believer that most apps should adhere to the system...it looks weird when a light app is alongside a bunch of dark ones, or vise versa). EDIT: Ok I do prefer dark mode, but it's a reward for working late, non? EDIT: Also the WWDC app on iOS now supports both themes, might be cool to match it. |
Thank you for the experimentation, but we don't have any intention of supporting both themes. |
I couldn't reproduce this anymore as of #600. Anyone? |
The text was updated successfully, but these errors were encountered: