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

--- Comment #2 from Nate Graham <n...@kde.org> ---
Investigated a bit. We have two options:
1. Append "-symbolic" to the current name of the icon we're asking for in the
config window, which is the icon in the applet's JSON metadata
2. Stop using the JSON Metadata icon in the config window, and instead use the
icon that the applet is currently showing in the System Tray itself

The downside to option 1 is that it's not guaranteed to actually accomplish the
goal, because there may not be a symbolic version of the icon specified in the
JSON metadata available. We can guarantee this for 1st party applets when using
the Breeze theme but not in other circumstances.

The downside to option 2 is that the icon shown in the config window will match
the applet's active status, rather than just being a generic representation of
it. For example when Bluetooth is disabled, the Bluetooth icon is sort of fades
and desaturated in the System Tray, and this would appear in the config window
as well.

Another problem with option 2 is that currently, asking for the applet's icon
causes plasmashell to crash and I haven't figured out how to fix it yet.

The code change in both cases is quite small, just a little change for
Qt::DecorationRole in plasma-workspace/applets/systemtray/systemtraymodel.cpp.
We just need to decide which one to do (and if we choose option 2, fix the
crash).

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

Reply via email to