Album-colored tint background in full view - #345
jojo-chaechae wants to merge 3 commits into
Conversation
Add an optional translucent tinted background derived from the album cover average color, with configurable opacity, bright album-derived text color, and a manual toggle to use the album-derived text color.
|
this is useful for kwin effects like blur (in the attached video, glass) , built on top of the existing transparency mode Screencast_20260813_120630.mp4 |
When the tint background is enabled, optionally use a vertical gradient derived from the album cover average color instead of a flat color.
|
added a gradient option rather than just a flat color, inspired by #244 |
There was a problem hiding this comment.
Thank you for the PR! I like the feature and it looks very good. Probably better than Use album cover as background.
I have only some concerns about the config UX. Now you can select both Use album cover as background and Use album cover color as tinted background simultaneously, but I think it's wrong.
We could place all the 5 options in a radio group, something like this:
Where the options background tint opacity and Use album-derived gradient are active only when the Use album cover color as tinted background is selected. While the option Use album-derived text color is active for both Use album cover as background and Use album cover color as tinted background.
What do you think?
|
Yes folding the album cover as an additional radio option would be simpler, wasnt sure what to do with it before so ignored it. Fixing now |
2 participants
Adds a translucent tinted background from the album cover color with configurable opacity, bright album-derived text, and a toggle for using the album-derived text color.