-
Notifications
You must be signed in to change notification settings - Fork 329
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
Dark theme #880
base: master
Are you sure you want to change the base?
Dark theme #880
Conversation
…l for light and dark themes. Also changed Theme to Theme.DayNight
Could you please rebase it to the latest master? Your branch unfortunately based on older code, and it seems that both branch contains changes in the same files, so automatic rebase is not possible. I tried to understand what went into your branch, but it looks strange - for example this commit: 94cf6db - it looks you cherry-picked a lot of recent change, and this merge: 1f19e42 - but it's hard to follow. |
I can rebase it, and fix the compilation error here |
@sanskar009, @gzsombor - IDK what's going on with this, but I only need the settings (preferences) not to be in White... |
IDK neither, I haven't worked on, after I tried to run, and rebased them. It's not too polished yet. |
SkyTube seems to work okay with Android's dark theme. Is this still needed? |
@hamishmb Forced (from Developer Options or with Shizuku and DarQ) ?, Yes, it is, because that way some elements are dark with dark text, making them unreadable. |
No, just checked and I don't have that option enabled. I'm on Android 11, if that makes any difference? I haven't noticed anything unreadable - which things display poorly for you? |
@hamishmb |
For some reason, I don't have that problem on mine. However, I have heard of vendor-dependent Android bugs, so if it affects you, it almost definitely affects others too. For reference I'm on a Moto G8 Power with Android 11. |
The whole point here is the settings/preferences menus to be in dark gray (/grey) or true amoled/oled black, along with other UI/layout elements, instead of white, like currently. |
Ah yes, I can see the white background when I open preferences, just not in the background of the "..." menu. Agreed that it's a worthwhile addition. |
No description provided.