[Icon request] MATE Sensors Applet

[ilya-fedin]

MATE Sensors Applet has several icons that represent some hardware:
https://github.com/mate-desktop/mate-sensors-applet/blob/66f1050a7da8d68494c3cd74e0f9601419f3e13a/sensors-applet/sensors-applet.h#L41-L50

Also, there are overlay icons that represent temperature: https://github.com/mate-desktop/mate-sensors-applet/blob/4afada78ef42b1e9ad044badf1c6b2b7758c105c/sensors-applet/active-sensor.c#L49-L55
However, I'm not sure if they could be themed since they're placed in /usr/share/pixmaps

Reference icons: https://github.com/mate-desktop/mate-sensors-applet/tree/a2c20a1a36600389e347f2cab5f87b95cbbb6eb5/pixmaps

[varlesh]

@ilya-fedin You can test (hardcoded or not) overlay icons. Create some symlinks to any file

[ilya-fedin]

They aren't hardcoded
https://github.com/mate-desktop/mate-sensors-applet/blob/66f1050a7da8d68494c3cd74e0f9601419f3e13a/sensors-applet/sensors-applet.c#L804

[SmartFinn]

@ilya-fedin the question about these icons https://github.com/mate-desktop/mate-sensors-applet/blob/4afada78ef42b1e9ad044badf1c6b2b7758c105c/sensors-applet/active-sensor.c#L49-L55 https://github.com/mate-desktop/mate-sensors-applet/blob/4afada78ef42b1e9ad044badf1c6b2b7758c105c/sensors-applet/active-sensor.c#L234

[varlesh]

confuses us PIXMAPS_DIR

[varlesh]

anyway we can fix that on hardcode-tray

[varlesh]

@ilya-fedin You can show screenshot how looking sensor with overlay icon?

[varlesh]

i'm google it:

[varlesh]

@ilya-fedin overlay icon can be disabled?

[ilya-fedin]

the question about these icons

However, I'm not sure if they could be themed since they're placed in /usr/share/pixmaps

Unfortunately :(

You can show screenshot how looking sensor with overlay icon?

overlay icon can be disabled?

No, I can't see such a setting

[varlesh]

@ilya-fedin sensors icons changed with latetst git-version?

[ilya-fedin]

[varlesh]

@ilya-fedin We decide add colored dot on upper right corner. It's not overlay icons and informatively

[varlesh]

22px only.
Hardcode-tray updated

[ilya-fedin]

I just filled an issue to mate-sensors-applet
mate-desktop/mate-sensors-applet#106