search

lxqt / lxqt-qtplugin

LXQt Qt platform integration plugin
https://lxqt.github.io
GNU Lesser General Public License v2.1
24 stars 14 forks source link

SNI: don't create a SNI if support isn't available #6

Closed paulolieuthier closed 8 years ago

paulolieuthier commented 8 years ago

Keep support for the good old QSystemTrayIcon. Inspiration from here.

Discussion in lxde/lxqt-panel#241.

pmattern commented 8 years ago

Commit solves all problems "plugin-statusnotifier vs. plugin-tray" as described in lxde/lxqt-panel#241: if one of the plugins is available only applications will pick it much as they will prefer plugin-statusnotifier if both are available. Arch i686 and x86_64.

In my opinion GTM.

paulolieuthier commented 8 years ago

@pmattern thanks again.

palinek commented 8 years ago

One note: This solves the first try/creation. If statusnotifier is not available in time of icon creation the icon chooses the tray. But if the statusnotifier is created and the tray closed there will be no icon available. Or vice-versa if the statusnotifier is chosen and tray is created together with closing statusnotifier we loose our icon too. Don't know if this is a blocker or is worth to implement.

paulolieuthier commented 8 years ago

I'm aware of that. KNotifications's KStatusNotifierItem handles it. However, I don't think it's worth it. The old spec is deprecated and users won't be enabling/disabling the plugins.