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

Dark theme #880

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Dark theme #880

wants to merge 7 commits into from

Conversation

sanskar009
Copy link

No description provided.

@gzsombor
Copy link
Member

gzsombor commented Jan 8, 2021

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.

@gzsombor
Copy link
Member

gzsombor commented Jan 9, 2021

I can rebase it, and fix the compilation error here
My main issue is that the dark theme does not match the current theme, it has more and different shades of grey. I would rather see that default outlook stays the same. Could you fix it?

@drogga
Copy link
Contributor

drogga commented Feb 17, 2021

@sanskar009, @gzsombor - IDK what's going on with this, but I only need the settings (preferences) not to be in White...

@gzsombor
Copy link
Member

IDK neither, I haven't worked on, after I tried to run, and rebased them. It's not too polished yet.

@hamishmb
Copy link

hamishmb commented Aug 2, 2022

SkyTube seems to work okay with Android's dark theme. Is this still needed?

@drogga
Copy link
Contributor

drogga commented Aug 3, 2022

@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.

@hamishmb
Copy link

hamishmb commented Aug 3, 2022

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?

@drogga
Copy link
Contributor

drogga commented Aug 18, 2022

@hamishmb
on S20 FE with A11 and Dark Mode enabled, for some reason the context (3-dots) menu is in white with white text color, making it unreadable.

@hamishmb
Copy link

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.

@drogga
Copy link
Contributor

drogga commented Aug 25, 2022

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.
I would like to see the line dividers be brought back.

@hamishmb
Copy link

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.

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.

7 participants