manjaro / pamac

Graphical Package Manager for Manjaro Linux with Alpm, AUR, Appstream, Flatpak and Snap support
GNU General Public License v3.0
237 stars 42 forks source link

Update tray to support SNI spec #159

Closed pdf closed 8 years ago

pdf commented 8 years ago

Currently the tray does not work with Plasma. In the short term, this may be a bug in the plasma xembed proxy, however the current hacks to continue xembed support in Plasma/Unity will not stick around in the long term.

I believe StatusNotifierItem is supported by most desktops (GNOME Shell being the exception, though they no longer support xembed either, and GtkStatusIcon is deprecated), and AIUI libappindicator supports SNI while providing a fallback to xembed, so just switching to libappindicator from libnotify might be enough to sort this out.

guinux commented 8 years ago

implemented in d840b05

pdf commented 8 years ago

So fast, thx @guinux!

Menu and icon updates appear to work well, however left-click does not open the updater when updates are pending - probably needs another conditional.

guinux commented 8 years ago

see c394a74

pdf commented 8 years ago

:+1: working now, thanks @guinux.

guinux commented 8 years ago

In Manjaro, the appindicator tray is now splitted in another package named pamac-tray-appindicator.

pdf commented 8 years ago

Thanks, appears to work well.