spicetify / spicetify-themes

A community-driven collection of themes for customizing Spotify through Spicetify - https://github.com/spicetify/cli
MIT License
5.11k stars 1.27k forks source link

Font Color Not Adapting to High Contrast Backgrounds in StarryNight/Base Theme #1109

Closed violetto-rose closed 1 month ago

violetto-rose commented 2 months ago

I'm experiencing an issue where the font color doesn't adjust properly when using the StarryNight or Base theme in Spicetify. Specifically, when the background changes based on album artwork, the font color remains the same, even with high contrast backgrounds, making the text difficult to read.

Steps to Reproduce:

Use the StarryNight or Base theme in Spicetify. Play music with album artwork that has a light or dark background. Observe that the font color does not change to adapt to the background, leading to low readability.

Expected Behavior: The font color should automatically adjust to provide sufficient contrast with the background, ensuring the text remains readable regardless of the album artwork colors.

Screenshots: Screenshot 2024-09-01 044640 Screenshot 2024-09-01 044705 Screenshot 2024-09-01 044718

Specifics OS: Windows 11 Home Single Language

b-chen00 commented 2 months ago

The StarryNight theme doesn't change the background or font color based on the album artwork. When using the base color scheme, the background should always be a dark blue color with white text as shown below rather than what the album artwork is. What extensions, snippets, or apps are you using that could potentially affect this? image

b-chen00 commented 2 months ago

Nevermind, I see a problem image

violetto-rose commented 2 months ago

Oh sorry, I didn't see your mails. Also, the extensions I'm using currently are attached below. I tried using custom CSS from ChatGPT using CSS Editor with the code attached below, which of course didn't work. Screenshot 2024-09-22 154922 Screenshot 2024-09-22 154934 Screenshot 2024-09-22 154951 Screenshot 2024-09-22 155302

b-chen00 commented 1 month ago

Waiting for the pull request to be merged (see linked pull request above). Should fix visibility issues though let me know if it doesn't once the changes make its way through