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

Black themes are not correctly toned down #961

Closed
4 tasks done
hknutsen opened this issue Jan 14, 2025 · 8 comments
Closed
4 tasks done

Black themes are not correctly toned down #961

hknutsen opened this issue Jan 14, 2025 · 8 comments
Assignees
Labels
bug Something isn't working ui Related to app UI

Comments

@hknutsen
Copy link

hknutsen commented Jan 14, 2025

Description

After upgrading to 4.0.0-dev3, I no longer get a pure-black background on the "Now playing" screen when the pure-black theme is enabled.

Library screen with pure-black theme enabled:

Library screen with pure-black theme enabled

"Now playing" screen with pure-black theme enabled (background is not pure-black):

"Now playing" screen with pure-black theme enabled

Library screen background color vs "Now playing" screen background color:

Background colors comparison

Evidence that pure-black theme ("Svart drakt" in Norwegian) is enabled:

bilde

Problem solved

No response

Other implementations

Other music players (Poweramp, Retro Music Player, Oto Music) enable a pure-black background on the "Now playing" screen when the pure-black theme (or their equivalent feature) is enabled.

Benefit

Improves visual consistency when pure-black theme is enabled.

Duplicates

@hknutsen hknutsen added the enhancement New feature or request label Jan 14, 2025
@Richard38907
Copy link

Same here, please re-add pure black player when "pure black" option is set to on.

@OxygenCobalt
Copy link
Owner

The new M3 guidelines require the use of these elevated colors, so the most I can do is tone them down. That requires me to regenerate most of my themes, I think. Will look into this, probably won't be 4.0.0.

@OxygenCobalt OxygenCobalt changed the title Use pure-black theme on "Now playing" screen in 4.0.0 Used toned-down black themes Jan 15, 2025
@OxygenCobalt OxygenCobalt added the ui Related to app UI label Jan 15, 2025
@TempUser13
Copy link

Would you consider doing the M3 black theme and a non M3 black theme / adding something like a 0 contrast/depth option (removing the grey parts) because it is kinda misleading having the grey colors there which isn't an attack on you or your choice to do M3 but it will be a thing stopping people upgrading and getting all of the benefits of v4.

@OxygenCobalt
Copy link
Owner

Since this seems quite in demand, I'm reclassifying this was a regression and going to focus on adding it to a patch release.

@OxygenCobalt OxygenCobalt added bug Something isn't working and removed enhancement New feature or request labels Feb 24, 2025
@OxygenCobalt OxygenCobalt changed the title Used toned-down black themes Black themes are not correctly toned down Feb 24, 2025
@OxygenCobalt
Copy link
Owner

0 contrast/depth option (removing the grey parts)

I mean, I could make the sheet backgrounds use outlines. This might be the easiest way to do """black mode"""

@TempUser13
Copy link

Thank you for reclassifying this as a regression and looking to "fix" it.

@OxygenCobalt
Copy link
Owner

New toned down black theme: Auxio_Canary.zip @TempUser13 @hknutsen @Richard38907

This is the most I'm willing to tone it down, any further and it starts colliding against the dynamic color options I have.

@hknutsen
Copy link
Author

hknutsen commented Mar 6, 2025

Looks much better to me, @OxygenCobalt. Thanks! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui Related to app UI
Projects
None yet
Development

No branches or pull requests

4 participants