https://bugs.kde.org/show_bug.cgi?id=353819

--- Comment #22 from Nate Graham <n...@kde.org> ---
There is no override; the issue here is the *lack* of an override. :) Right now
the icon colors take into account only Plasma's own color scheme and therefore
when using a light theme you get black icons, and when using a dark theme you
get white icons. However it doesn't take into account the background color of
the note itself. The UI elements of the note would have to be taught about
that.

The way this works is that the icons' colors are hardcoded into the SVG, but
can be re-colored at runtime by code in Plasma that reads some CSS inside the
icons. However it only exposes the ability to choose which color set the color
comes from; you can't just shim in an arbitrary color. So you can say "give me
the text color from the View color set", or from the window color set, or from
the header color set, or whatever. But that won't help much in this case
because the available color sets don't map to the note's available background
colors.

An easy fix for this would simply to put a window-colored rectangle under the
controls, so they get drawn on top of a background whose color they expect.
However this might be considered ugly. And it couldn't help the scrollbar
unless you did it for that too, which would be even more like to be ugly.

It's a bit of a tricky problem to solve for the icons. For text, it's easier
since you can actually force a specific color. So we could simply store a
mapping of appropriate colors for each note color and use those for the text.
But you can't currently do this for the icons, which is the problem.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to