Bug 348155 - Limitation in the AppIndicator support
Summary: Limitation in the AppIndicator support
Status: CLOSED INTENTIONAL
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray (show other bugs)
Version: 5.3.0
Platform: Kubuntu Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL: https://groups.google.com/d/msg/psens...
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-23 15:32 UTC by jeremy9856
Modified: 2015-06-18 23:02 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description jeremy9856 2015-05-23 15:32:06 UTC
I installed psensor app-indicator and the problem is that the temps can't be displayed in systray. The psensor developer say this is a limitation of the AppIndicator support of KDE.

https://groups.google.com/d/msg/psensor-users/Uk1DWghoHXQ/fnyNFk2Ax_IJ

Reproducible: Always

Steps to Reproduce:
1. Install psensor
2. Try to enable temp in systray

Actual Results:  
nothing happen

Expected Results:  
temp are displayed in systray
Comment 1 David Edmundson 2015-05-29 15:32:31 UTC
I installed it, and took the dbus logs:

it is emitting a signal with the member
org.kde.StatusNotifierItem.XAyatanaNewLabel
with the arguments  "56\176C"  "999UUU" 

but that signal doesn't exist. In the spec:
http://www.freedesktop.org/wiki/Specifications/StatusNotifierItem/StatusNotifierItem/

there is no XAyantanaNewLabel signal.

I don't know where the psensor author got that from. I'd be interested to know

I guess it might be a Ubuntu extension? But if so it definitely should *not* be under our org.kde namespace. 
(which btw, you should upgrade to org.freedesktop at some point)

Can you copy and paste this back on that original Google groups page. I can see responses there, but I don't have an account there. 
Sorry to keep bouncing you round between bug trackers.
Comment 2 jeremy9856 2015-05-29 20:09:55 UTC
I did it, let me know it you need something else.
Comment 3 jeremy9856 2015-06-01 13:54:31 UTC
I don't know if that can help but I found this :

http://people.canonical.com/~dpm/api/stable/libappindicator/libappindicator-app-indicator.html
Comment 4 jeremy9856 2015-06-17 21:48:16 UTC
The link in my previous comment does not help ?
Comment 5 David Edmundson 2015-06-18 22:47:43 UTC
It helps explains where the psensor guy got it from, but I don't understand why Ubuntu have just added stuff to things that are clearly not in the original specification without telling us.

I don't understand what on Earth Ubuntu think they're doing. 

The definitive spec is here: http://www.freedesktop.org/wiki/Specifications/StatusNotifierItem/
If they don't add things there their random extensions can't be supported. Period.

(sorry)
Comment 6 jeremy9856 2015-06-18 23:02:16 UTC
Ok I understand. Ubuntu did again something on his side...
I really like Psensor but we can live without the incompatible appindicators, we have the plasmoids instead.

If you are in touch with the Ubuntu devs, it can be interesting to ask them why we are in this situation now ?