ubuntu / gnome-shell-extension-appindicator

Adds KStatusNotifierItem support to the Shell
https://extensions.gnome.org/extension/615/appindicator-support/
GNU General Public License v2.0
1.21k stars 163 forks source link

Legacy X11 tray-icons no longer work after changing "Tray horizontal alignment" #302

Open jwrdegoede opened 3 years ago

jwrdegoede commented 3 years ago

When changing the "Tray horizontal alignment" option in the preferences something weird happens with legacy X11 tray-icons. They can no longer be clicked, not only that but the highlight on mouse over only gets shown when hovering the cursor over the highlight-area, when the cursor is over the actual icon, the icon also is not highlighted.

But things get even more weird, it does not matter to which value the "Tray horizontal alignment" option is changed, changing it once causes this broken behavior, changing it a second time fixes things, changing it a third time breaks things again, a fourth change fixes things again, etc.

So one can change from say right to center alignment with things breaking by doing the following:

  1. left -> right
  2. right -> center

Where as directly going from right to center breaks things (until the user logs-out + in again then things are normal again.

I've seen this behavior with topicons-plus too and this is likely a gnome-shell or mutter issue, I had a bug open against either gnome-shell or mutter on bugzilla.gnome.org, but I cannot find that bug and IIRC the bug never got any comments so it is not important.

Even though as I said I believe that this is likely a gnome-shell/mutter issue it seems best to track it here, because this is the most likely place for people to look for tray-icon issues.

jwrdegoede commented 3 years ago

p.s.

As already mentioned in #298 thank you for adding legacy X11 tray-icon support. As issues 298-302 show the legacy X11 tray-icon support unfortunately has various issues/warts. Most of those may have their root cause outside of the appindicator extension, but it seemed best to file / document them all in one place.

Perhaps a section with "known legacy X11 tray-icon issues" should be added to the README, combined with a text advertising users to try and get any apps still using this to using appindicator ?