Summary: | Can't remove Discover Update Notifier from System Tray | ||
---|---|---|---|
Product: | [Applications] Discover | Reporter: | Sebastian Friedl <sfr682k> |
Component: | Notifier | Assignee: | Aleix Pol <aleixpol> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | aleixpol, cappelikan, kde, nate, piotr.mierzwinski, postix, priit, putr4.s, qydwhotmail, rdieter, rikmills, s1r51ckb01 |
Priority: | NOR | Keywords: | usability |
Version: | 5.27.7 | ||
Target Milestone: | --- | ||
Platform: | Neon | ||
OS: | Linux | ||
See Also: |
https://bugs.kde.org/show_bug.cgi?id=416193 https://bugs.kde.org/show_bug.cgi?id=454422 |
||
Latest Commit: | https://invent.kde.org/plasma/discover/-/commit/0ab7b2c87fcbb4c144fcea917809b8dd67f35d0d | Version Fixed In: | 6.0 |
Sentry Crash Report: |
Description
Sebastian Friedl
2019-10-16 19:21:45 UTC
Yeah, this is a side effect of it being an executable rather than a system tray applet. (In reply to Nate Graham from comment #1) > Yeah, this is a side effect of it being an executable rather than a system > tray applet. Hi Nate first of all congratulations as always for the excellent work you do to create and make always better Kde. I would like to ask you how you could disable this behavior? The "/usr/lib/DiscoverNotifier" process starts with "CGroup: /user.slice/user-1000.slice/session-2.scope" You should see a combo box under "Entries" which can be set to Hidden. Does this work? I think Sebastian wants the ability to remove it entirely, not just banish it to the expanded popup. (In reply to Nate Graham from comment #4) > I think Sebastian wants the ability to remove it entirely, not just banish > it to the expanded popup. That's right. I've disabled the expanded popup so I could see all tray icons. Setting Discover's entry to "Hidden" would require to disable the "Show all entries" option and set all other entries to "Always shown". It is of course possible to right click the icon, and select 'quit'. However, I guess the biggest issue here is that there is no GUI way to prevent the DiscoverNotifier autostarting with plasma as could be done when it was an applet. There is no way to do it via the system tray, and the notifier does not show up in any class of startup items in systemsettings. A way to disable it forever is to run something like: $ cp /home/apol/devel/kde5/etc/xdg/autostart/org.kde.discover.notifier.desktop ~/.local/share/autostart $ echo "Hidden=true" >> ~/.local/share/autostart/org.kde.discover.notifier.desktop I can look into adding some UX to make that happen for 5.18, it's not really possible to do it right without new strings. Git commit 87e145613d161dc9ebf2ab1fbc049840c2b87e2a by Aleix Pol. Committed on 23/10/2019 at 16:29. Pushed by apol into branch 'master'. notifier: make it possible to have the notifier hidden But still regularly check for updates M +1 -1 notifier/CMakeLists.txt M +0 -2 notifier/DiscoverNotifier.cpp A +106 -0 notifier/NotifierItem.cpp [License: GPL (v2/3)] A +46 -0 notifier/NotifierItem.h [License: GPL (v2/3)] M +25 -60 notifier/main.cpp https://commits.kde.org/discover/87e145613d161dc9ebf2ab1fbc049840c2b87e2a As commented by Aleix Pol I solved by adding to the bottom of the file: /etc/xdg/autostart/org.kde.discover.notifier.desktop the string: Hidden=True When restarting, the icon does not appear in the system tray and the process does not start automatically Aleix, did your last commit fix this? Or is there still something left to do? (In reply to Nate Graham from comment #10) > Aleix, did your last commit fix this? Or is there still something left to do? No, as there is no way to _persistently_ disable it. This was possible to do when it was an applet. Running "/usr/lib/libexec/DiscoverNotifier --hide" should prevent it from showing up again. If you don't want it to start at all, its autostart needs to be disabled. If we want an in-between solution for the notification, we could add the possibility to silence it for an $amount-of-time. Or it could be something that could be added generically to KNotifications? (In reply to Aleix Pol from comment #12) > Running "/usr/lib/libexec/DiscoverNotifier --hide" should prevent it from > showing up again. > > If you don't want it to start at all, its autostart needs to be disabled. I don't see an entry for it in the autostart KCM. IMO we should probably expose this in Discover's own settings too, along with the frequency with which it checks for updates. having xdg autostart entries in autostart kcm module will be very useful, configurable update interval in discover is too (or even controling apt built-in update interval) Any progress on this? *** Bug 432439 has been marked as a duplicate of this bug. *** For those who use Arch based distributions. In below url you will find some work around how to fix this issue. Shortly: update of pacman.conf by adding a rule to skip desktop (org.kde.discover.notifier.desktop) file in moment of extracting of package. https://www.reddit.com/r/kde/comments/lavors/getting_rid_of_discover_keeps_popping_back_up/ FWIW I've discovered that openSUSE and Fedora are splitting the notifier into a separate package to allow users who don't want tho notifier to remove it. This is a pretty heavyweight solution, and I think we can make life easier for them and our users by making this an option native to Discover itself. It would probably make sense to do at the same time as Bug 416193; if the update interval is set to "Never", or there are simply no updates available, the Notifier could automatically disappear, for example. *** Bug 456069 has been marked as a duplicate of this bug. *** there was an error with the non-disappearing icon after the update, you have to forcefully close it manually Operating System: KDE neon 5.27 KDE Plasma Version: 5.27.7 KDE Frameworks Version: 5.108.0 Qt Version: 5.15.10 Graphics Platform: X11 I would like to see an option for actually DISABLING it. Being hidden is not good enough -- I don't want the process `/usr/lib/DiscoverNotifier` even launch. Ever. (while at the same time not uninstalling the entire Discover for achieving this) (In reply to Nate Graham from comment #19) > FWIW I've discovered that openSUSE and Fedora are splitting the notifier > into a separate package to allow users who don't want tho notifier to remove > it. In Plasma 5.27 this is also the case in K/Ubuntu, by removing the package plasma-discover-notifier. A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/721 A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/729 Git commit 0ab7b2c87fcbb4c144fcea917809b8dd67f35d0d by Yifan Zhu. Committed on 09/01/2024 at 17:31. Pushed by apol into branch 'master'. notifier: only show applet when necessary Only show the notifier applet when there are interesting things to notify about: - update notifications - pending reboot - and ongoing updates. M +51 -26 notifier/NotifierItem.cpp M +8 -4 notifier/NotifierItem.h M +1 -1 notifier/main.cpp https://invent.kde.org/plasma/discover/-/commit/0ab7b2c87fcbb4c144fcea917809b8dd67f35d0d |