andia89/order-icons

Legacy icons ever create a new entrace in extension configuration

Closed this issue · 6 comments

It is possible that legacy indicators and icons (such as those of Wine) are grouped all at once, because whenever I start another instance of the application, another entry is created in the extension's configuration menu.
Captura de tela de 2023-05-22 14-55-12
Thanks again for the excellent work on this extension.
image

Don't really manage to recreate this problem. Is that maybe with an older version of gnome-shell-appindicators extension? If not is it a specific app it happens with? If I can recreate it should be very easy to fix

Definitely a thing that still happens. Using the latest version from AppIndicator and KStatusNotifierItem Support for GNOME 45 and latest version from this extension as well. It continues to heavily pollute the list, plus it's basically impossible to keep those apps ordered long-term because a new line simply gets created and your old ordered ones get buried.

Basically without fixing this, the extension ends up serving me no purpose, as half of my tray icons are legacy ones that do this.

So far, happens with all appindicator-legacy apps. Apps I personally use that are affected: Fcitx5, KeePassXC, Trackma, Steam.

image

Maybe some sort of regex or pattern name matching could help, so that for example appindicator-legacy:KeePassXC:* could be ordered at index X?

Right it is very easy to fix I think, but I need to be able to reproduce it. Can you maybe tell me exactly which version of KeepasXC (appimage, ppa etc...) you have installed?

Running:

Those seem like the biggest offenders in legacy icon generation from my screenshot?

Possible reproduction steps (things that could re-initialize tray icons) could be any of:

  • Toggling the AppIndicator and KStatusNotifierItem Support on and off
  • Restarting GNOME with Alt+F2, although GNOME 45 crashes the entire session when doing that
  • Restarting the affected applications

Ok I'm on it. Managed to reproduce

fixed with 54d4058