Bug 352055 - plasma-pa plasmoid not shown in systemtray after startup
Summary: plasma-pa plasmoid not shown in systemtray after startup
Status: RESOLVED UNMAINTAINED
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray (show other bugs)
Version: 5.4.2
Platform: openSUSE Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
: 347554 352580 356555 356605 356705 356846 357803 358319 361126 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-08-31 12:12 UTC by Robby Engelmann
Modified: 2016-09-11 19:51 UTC (History)
61 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Missing Audio Volume indicator in systray (5.85 KB, image/png)
2015-10-27 09:22 UTC, René Krell
Details
half an icon (4.98 KB, image/jpeg)
2016-01-10 04:27 UTC, Mark
Details
clean boot (1.24 MB, image/png)
2016-01-14 22:55 UTC, Nikola Schnelle
Details
after clicking on systray arrow (1.23 MB, image/png)
2016-01-14 22:56 UTC, Nikola Schnelle
Details
hidden icon and entry (16.15 KB, image/png)
2016-02-06 21:05 UTC, Fahad Alduraibi
Details
gaps in system tray (13.31 KB, image/png)
2016-02-29 02:47 UTC, Sudhir Khanger
Details
missing systray icons (47.36 KB, image/jpeg)
2016-09-11 16:17 UTC, Hrtmt Brng
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Robby Engelmann 2015-08-31 12:12:03 UTC
the plasma-pa plasmoid sometimes do not show up in systemtray. Instead a "hole" (empty area) is shown, which is not clickable

Reproducible: Sometimes

Steps to Reproduce:
1. install plasma 5.4 and plasma-pa
2. start in plasma5 session

Actual Results:  
plasma-pa not shown and not interactable

Expected Results:  
plasma-pa should show every time after starting the session

workaround: go into systemtray setting, deactive audio volume --> apply --> activate audio volume --> apply
now plasmoid is shown
Comment 1 Ramon Buldó 2015-09-12 01:02:21 UTC
Had this problem too with 5.4.0 but with 5.4.1 seems to be fixed.
Comment 2 Albert Astals Cid 2015-10-08 17:31:26 UTC
I still have this problem with 5.4.2
Comment 3 Shiba 2015-10-14 07:20:17 UTC
The problem somehow changed for me with frameworks 5.15 and plasma 5.4.2: now plasma-pa starts correctly and the network-manager plasmoid is showing the same problem instead.
Comment 4 Matt 2015-10-26 14:02:59 UTC
Plasma-pa intermittently causes the same issue for me running 5.4.2 on Kubuntu 15.10. Cycling it in systray settings works as stated.
Comment 5 René Krell 2015-10-27 09:22:17 UTC
Created attachment 95159 [details]
Missing Audio Volume indicator in systray

Missing Audio Volume indicator after logon also on OpenSUSE Tumbleweed 20151022, using
- KDE Releases Applications 15.08.2
- KDE Releases Frameworks 5.15.0
- Plasma 5.4.2
- Qt 5.5.1

Workaround as already described: System Tray Settings -  General - Uncheck Extra Items -> Audio Volume, press Apply, recheck Extra Items -> Audio Volume, press Apply.
Comment 6 hegemon8 2015-10-27 09:39:52 UTC
Same as in Comment 5.
Empty space instead of audio icon after every reboot. 
KDE Plasma Version: 5.4.2
QT Version: 5.4.2
Kernel Version: 4.2.0-16-generic
OS: Kubuntu 15.10  64-bit (Breeze Dark Theme)
Same workaround works as described above.
Comment 7 Darryl 2015-10-27 22:54:44 UTC
This affects me too on Kubuntu 15.10 64bit. Currently using the Oxygen theme.
Plasma 5.4.2
qt 4.5.2
Kernel 4.2.4
Comment 8 Robby Engelmann 2015-10-28 06:02:40 UTC
also valid using openSUSE snapshot 20151022 with QT 5.5.1
Comment 9 Gauvain Roussel-Tarbouriech 2015-10-29 08:05:56 UTC
Yup affects me as well on Kubuntu 15.10 64-bits, Plasma 5.4.2
It worked like once or twice as far as I remember then got a white space instead of plasma-pa
Comment 10 Viorel-Cătălin Răpițeanu 2015-10-31 07:43:42 UTC
I got the same problem on Arch Linux.
An easy workaround would be:
- Configure the tray manager.
- Disable the audio applet then Apply.
- Enable the audio applet then Apply.

Even if there is an woking workaround for this, in my opinion this must be fixed for the next release.
Comment 11 Nils 2015-11-02 13:57:47 UTC
I can confirm this issue on Archlinux too.
Comment 12 Albert Astals Cid 2015-11-02 21:47:05 UTC
After spending most of the weekend debugging it i am quite convinced this is caused by an upstream bug in Qt https://bugreports.qt.io/browse/QTBUG-48870

We are trying to get this fixed but understand it may take a bit.
Comment 13 Sergio 2015-11-09 11:54:32 UTC
Fortunately, the issue is easily worked around by uninstalling plasma-pa and installing kmix.

Still, the question is: even if this is a qt bug, why pushing plasma-pa in place of kmix that used to work just fine, before checking if plasma-pa could work fine?  It seems impossible to me that this could have gone unnoticed by anyone who used plasma 5 for at least 2 hours in a row.

As a matter of fact, it looks like almost every single serious plasma bug is in fact a bug in Qt (e.g. impossibility to work with projectors or external screens in a reliable way, this one, etc.). Still it would be very much appreciated if KDE could test changes against the versions of Qt that actually one can find in distros before pushing them in releases.
Comment 14 Albert Astals Cid 2015-11-10 23:24:50 UTC
> before checking if plasma-pa could work fine? It seems impossible to me that this could have 
> gone unnoticed by anyone who used plasma 5 for at least 2 hours in a row.
As impossible at it seems to you, it's the truth. This bug only manifests on machines with slow hard drives, since most of the developers use fast SSD drivers they did not see it.

> Still it would be very much appreciated if KDE could test changes against the versions of Qt that 
> actually one can find in distros before pushing them in releases.
Let me remind you that KDE is mostly a volunteer driven organization, all the help you can provide testing will make it better for the fellow users.
Comment 15 Rex Dieter 2015-11-10 23:57:53 UTC
And pre-emptive,
https://www.kde.org/code-of-conduct/
as a reminder to be considerate, respectful, supportive (as well as collaborative and pragmatic)
Comment 16 Matt Scheirer 2015-11-10 23:59:27 UTC
(In reply to Albert Astals Cid from comment #14)
> > before checking if plasma-pa could work fine? It seems impossible to me that this could have 
> > gone unnoticed by anyone who used plasma 5 for at least 2 hours in a row.
> As impossible at it seems to you, it's the truth. This bug only manifests on
> machines with slow hard drives, since most of the developers use fast SSD
> drivers they did not see it.
> 

This bug manifests on fresh installs of Kubuntu 15.10 on two systems with a Samsung 830 and a Corsair Force LS SSD respectively. Its not limited to slow hard drives.
Comment 17 Nils 2015-11-11 01:20:09 UTC
I am also using SSD drives but still have the issue.
Comment 18 Robby Engelmann 2015-11-11 06:09:17 UTC
I can confirm that, I am using a Lenovo X220 with SSD.
Comment 19 Sergio 2015-11-11 09:45:19 UTC
I am constantly seeing the issue on a laptop with SSD disk exactly in the same way in which i constantly see it on a desktop with a rather slow software raid built on rotating media. Makes no difference at all.  The only difference that I see is that the desktop machine takes 2 minutes from the complete filling of the startup progress bar to getting an environment with all the plasmoid populated (evidently hanging on some plasmoid), while the laptop does this in a few seconds. 

Incidentally, my remark was referring to the fact that for almost every single bug I opened about plasma 5 either here or more frequently on ubuntu launchpad (and they were more than 20 in the first two hours of use), I found that the issue was in Qt and, often, in the specific version of Qt found in my distro. While this casts some shade on the reliability of Qt as a platform, I think that it would be very important to assure that before pushing out a new feature in plasma it is tested against the libraries that can actually be found in distros. If the feature does only work reliably on developers machines that may have some extra-new in-development version of Qt and not with the distro version of Qt, my suggestion is to leave it cooking for later.  Releasing it immediately means that users will hit against bugs in Qt on their platform and attribute the issue to KDE, with a huge damage to the KDE reputation.  

For the issue at hand, to me this boils down to reverting to kmix (that works just fine) and keep plasma-pa on hold, until distros can catch up with a version of Qt that can support plasma-pa without issues.  

I thought that this could have been considered a constructive suggestion. If it is not, or if it is against the KDE code of conduct, please take my apology and feel free to ignore it.
Comment 20 Albert Astals Cid 2015-11-11 19:40:40 UTC
I should have used a better working, disk speed is one of the contributing factors that makes this bug come or go away, obviously not he only one, also number of items on the systray, etc have a factor of influence on whether the bug manifests itself or not.

For example, on each boot i will have the bug, if then go to a shell, do
    killall plasmashell; plasmashell &
it will work fine. And that's basically restarting plasmashell from scratch, so yes, it is not as easy to reproduce as people that have it all the time think.
Comment 21 David Edmundson 2015-11-11 22:01:42 UTC
I have a machine where it does happen repeatedly. 

Qt model test on the proxy shows everything is fine

A manual inspection of all the changes all look sensible

That warning about anchors is unrelated. If I fix that this problem remains.

Setting a giant cache buffer should mean all delegates are "in view" at any point. This fixes the test in the linked Qt report perfectly, but it doesn't fix our case, which implies ours is subtly different to the Qt bug report.

Setting the grid size to a fixed huge size (so delegates are always in view) doesn't fix anything

Nor does making sure delegates have a fixed size

Swapping from GridView to Grid/Flow + Repeater causes a crash in Qt. Possible easier to debug. Possibly a different issue.

Adding a shoddy timer delay before loading the applets might be enough, but I don't want to go down that route.
Comment 22 Piotr Dobrogost 2015-11-18 14:53:20 UTC
I'm observing the same bug but in my case it's Instant Messaging applet.

Fedora 22
plasmashell 5.4.3 
Qt 5.5.0
Comment 23 Clément S 2015-11-21 13:58:58 UTC
Same bug over here with Audio volume. Workaround must be applied at each start-up !
Comment 24 Nick 2015-11-28 18:33:14 UTC
I have this same problem but with the bluetooth icon. Should I open a separate bug?
Comment 25 Albert Astals Cid 2015-12-07 20:43:36 UTC
Nick, no you should not.

As I already mentioned this is a bug in Qt and has been fixed for Qt 5.6 with https://codereview.qt-project.org/#/c/141620/

I suggest you guys contact your distributions and let them know about the existance of this bug and ask for a backport of the patch to the Qt they ship if you think this is critical enough.
Comment 26 Sergio 2015-12-07 21:28:07 UTC
I would not hold my breath waiting for distros to backport things or apply patches downstream E.g., the issue with hplip not being able to work on plasma 5 has been having a patch since 2014-07-29 thanks to Harald Sitter, but still, after more than 1 year the patch is not applied in (K)ubuntu...  Is there any news of some distro picking up Qt 5.6 for their next release? Will (K)ubuntu 16.04 be 5.6 or 5.5.x?
Comment 27 Albert Astals Cid 2015-12-07 21:30:16 UTC
This is really not the place to discuss about what KUbuntu or any other distro will or will not do.
Comment 28 Weng Xuetian 2015-12-09 05:50:39 UTC
Hi, I patched our qt5 with patch above (together with some other necessary changes) ,  but still have the same problem. Can you confirm it is the same issue? (Or, did you try the qt 5.6 branch?)
Comment 29 Albert Astals Cid 2015-12-09 21:25:59 UTC
Yes, i did try the patch and it did seem to fix the problem for me. I tried the patch on top of my heavily patched Qt 5.4
Comment 30 Weng Xuetian 2015-12-09 23:38:01 UTC
When I clicked on the "hole" on my system, I can see some warning from plasmashell input, which says "modelData" undefined (qml file is PlasmoidItem.qml). And I tried to print out the modelIndex, and found it's -1.

So this is not a visual problem but the attached model infomation is wrong. While the linked QT bug itself looks like a pure visual problem. I'm afraid that QT-BUG 48870 is not the only problem.
Comment 31 Leszek Lesner 2015-12-13 12:08:46 UTC
@Weng Xuetian
also patched it here on Qt 5.5.1 but still getting the same error. 
So patching Qt alone does not fix it apparently.
Comment 32 Albert Astals Cid 2015-12-13 21:32:20 UTC
I could always reproduce this bug doing
       echo 3 > /proc/sys/vm/drop_caches
as root and then as my user killing and restarting plasmashell.

But now I can not reproduce it anymore. Neither in my two Ubuntu Xenial systems running  Plasma 5.4.3 + Qt 5.5.1 nor on my Arch Linux system running Plasma 5.5.1 + Qt 5.5.1 so can't help with fixing this anymore.

If you guys are convinced the above patch does not fix it reopen the bug.
Comment 33 Leszek Lesner 2015-12-13 22:43:06 UTC
(In reply to Albert Astals Cid from comment #32)
> ...
> 
> But now I can not reproduce it anymore. Neither in my two Ubuntu Xenial
> systems running  Plasma 5.4.3 + Qt 5.5.1 nor on my Arch Linux system running
> Plasma 5.5.1 + Qt 5.5.1 so can't help with fixing this anymore.
> 
> If you guys are convinced the above patch does not fix it reopen the bug.

Can you attach your patch for Qt 5.5.1 so I can check with mine. Maybe I missed something important. (and others too) 
And did you really mean Plasma 5.5.1 ? Does it add a delay or other changes that 5.5.0 maybe does not have to starting the tray or icons or plasma-pa ?
Comment 34 Albert Astals Cid 2015-12-13 23:43:30 UTC
You misunderstood me, I can't reproduce the problem even without the patch.
Comment 35 Albert Astals Cid 2015-12-13 23:56:40 UTC
(In reply to Albert Astals Cid from comment #34)
> You misunderstood me, I can't reproduce the problem even without the patch.

To be extra clear, I could before, but I can't anymore, hence the "I can't help with fixing this anymore."
Comment 36 Weng Xuetian 2015-12-14 06:16:35 UTC
(In reply to Albert Astals Cid from comment #32)
> I could always reproduce this bug doing
>        echo 3 > /proc/sys/vm/drop_caches
> as root and then as my user killing and restarting plasmashell.
> 
> But now I can not reproduce it anymore. Neither in my two Ubuntu Xenial
> systems running  Plasma 5.4.3 + Qt 5.5.1 nor on my Arch Linux system running
> Plasma 5.5.1 + Qt 5.5.1 so can't help with fixing this anymore.
> 
> If you guys are convinced the above patch does not fix it reopen the bug.

Personally I don't have problem on my system with 5.4.3. I tried to bisect the problem, and actually found it's a007f294b4991c6f5730e0ec8d9d20f36e31acab, which is quite unrelated to the original problem. More specifically, it's this change below. But anyway it seems to be a Qt bug since if I increase the time (0 -> 300), then I will be able to reproduce the bug will be it no matter which syntax it is used  on my system.

diff --git a/applets/systemtray/plugin/host.cpp b/applets/systemtray/plugin/host.cpp
index 27ca227..61e8705 100644
--- a/applets/systemtray/plugin/host.cpp
+++ b/applets/systemtray/plugin/host.cpp
@@ -99,7 +99,7 @@ Host::Host(QObject* parent) :
     QObject(parent),
     d(new HostPrivate(this))
 {
-    QTimer::singleShot(0, this, SLOT(init()));
+    QTimer::singleShot(0, this, &Host::init);
 }

Now I workaround the bug by using "Always show all icon" option in system tray.

BTW To reproduce this bug, I need to have at least 10 icons in the system tray.
Comment 37 Leszek Lesner 2015-12-14 09:56:52 UTC
(In reply to Weng Xuetian from comment #36)
> ...
> 
> diff --git a/applets/systemtray/plugin/host.cpp
> b/applets/systemtray/plugin/host.cpp
> index 27ca227..61e8705 100644
> --- a/applets/systemtray/plugin/host.cpp
> +++ b/applets/systemtray/plugin/host.cpp
> @@ -99,7 +99,7 @@ Host::Host(QObject* parent) :
>      QObject(parent),
>      d(new HostPrivate(this))
>  {
> -    QTimer::singleShot(0, this, SLOT(init()));
> +    QTimer::singleShot(0, this, &Host::init);
>  }
> 

You are my personal hero man ;) 
Yes indeed reverting that patch fixes the issue for me too. 
So changing from Plasma 5.5 defaults
> QTimer::singleShot(0, this, &Host::init);
to 
> QTimer::singleShot(0, this, SLOT(init()));
and I get my plasma-pa icon on every boot :)
Comment 38 David Edmundson 2015-12-14 20:00:04 UTC
At first these look identical, but on closer inspection of QTimer:

void 	singleShot(int msec, Qt::TimerType timerType, const QObject * receiver, const char * member)
void 	singleShot(int msec, const QObject * receiver, PointerToMemberFunction method)

behave differently when msec == 0
the first version has a shortcut path that does a QMetaObject::invokeMethod with QtQueuedConnection.

The latter does not and goes via an actual timer; which possibly means it's called directly? 

Definitely not the true "cause" of the bug, but I'm willing to believe this has a significant impact on whether we hit the bug or not.

+1 to merging this.
Comment 39 David Edmundson 2015-12-15 12:54:29 UTC
Git commit a0578e6e5dbd429366e6a9837af543a4c85b5a75 by David Edmundson, on behalf of Xuetian Weng.
Committed on 15/12/2015 at 12:54.
Pushed by davidedmundson into branch 'Plasma/5.5'.

Workaround a system icon tray issue in bug

QTimer::singleShot(time, QObject*, char*)
QTimer::singleShot(time, QObject*, PointerToFunction)

behave differently when time == 0

So the porting had an effect

This patch migrates to QMetaObject::invokeMethod which is clearer

There seems to be two different bug in 352055 but looks similar, the other one is fixed in Qt 5.6 according to Albert Astals Cid.

M  +3    -2    applets/systemtray/plugin/host.cpp

http://commits.kde.org/plasma-workspace/a0578e6e5dbd429366e6a9837af543a4c85b5a75
Comment 40 David Edmundson 2015-12-15 13:00:10 UTC
*** Bug 356605 has been marked as a duplicate of this bug. ***
Comment 41 David Edmundson 2015-12-15 13:00:19 UTC
*** Bug 356555 has been marked as a duplicate of this bug. ***
Comment 42 David Edmundson 2015-12-15 13:01:01 UTC
*** Bug 352580 has been marked as a duplicate of this bug. ***
Comment 43 David Edmundson 2015-12-15 13:01:16 UTC
*** Bug 347554 has been marked as a duplicate of this bug. ***
Comment 44 tromzy 2015-12-15 13:45:11 UTC
Does this bug affect the plasma-pa icon only, or any icon in the systray ? Because it happens on my machine with plasma-pa, and it just happened with the kde-connect icon too.
Comment 45 Matt Scheirer 2015-12-15 14:22:08 UTC
It also happens with the Media Player. If both are active at boot the Media Player doesn't show, if Media Player isn't active pa doesn't show.
Comment 46 David Edmundson 2015-12-15 14:23:56 UTC
it affects the systray in general not just plasma-pa.

Plasma PA just seemed to get hit the most often; probably has the slowest startup time or something.
Comment 47 Bhushan Shah 2015-12-15 16:05:35 UTC
*** Bug 356705 has been marked as a duplicate of this bug. ***
Comment 48 Olivier Churlaud 2015-12-19 11:42:35 UTC
The problem disappeared and came back... Again with plasma-pa.

My system:
Archlinux
Plasma 5.5.1  (on Arch, the package is  plasma-workspace 5.5.1.1-1)
Frameworks 5.17.

I reopened... Feel free to tell me to open a new bug if needed.
Comment 49 Olivier Churlaud 2015-12-19 11:43:19 UTC
Killing and starting again plasmashell solves the issue...
Comment 50 Kyrylo Bohdanenko 2015-12-19 11:51:57 UTC
(In reply to Olivier Churlaud from comment #49)
> Killing and starting again plasmashell solves the issue...

You can also try toggling off and on plasma-pa widget in systemtray settings
Comment 51 Christoph Feck 2015-12-20 02:53:33 UTC
*** Bug 356846 has been marked as a duplicate of this bug. ***
Comment 52 Mark 2015-12-20 03:57:03 UTC
definitely not just plasma-pa
while plasma-pa is not there on almost every reboot my ibus icon is also not shown - but much more randomly - probably on about 50% of reboots it isn't there.
ibus is part of the "application status" part of sys tray so toggling that setting gets it back.
I've observed this bug consistently from plasma 5.4.2 through to 5.5.1 on openSUSE Leap
Comment 53 David Edmundson 2015-12-27 19:35:07 UTC
Anything changed with Plasma 5.5.2?
Comment 54 Matt Scheirer 2015-12-27 20:09:26 UTC
On Arch this bug is still happening consistently across three computers all running 5.5.2. Its always the media player or volume applets that go missing. Stock everything, so no qt 5.6 git or anything.
Comment 55 Olivier Churlaud 2015-12-28 17:53:37 UTC
ArchLinux, Plasmashell 5.5.2, bug is still there.
Comment 56 Mark 2016-01-10 04:27:52 UTC
Created attachment 96559 [details]
half an icon

so plasma 5.5.3 is out and this bug appears to be getting worse - not better.
Up until now the klipper icon in the tray has never been effeced - but now in 5.5.3 on every boot I am getting less than half a klipper icon.
toggling system services in tray settings restores a full icon.
Comment 57 Geoffrey Bonneville 2016-01-14 21:43:11 UTC
Same here on two archlinux machines with plasma 5.5.3
Comment 58 Nikola Schnelle 2016-01-14 22:52:16 UTC
After updating to KDE Framework 5.18 in Manjaro system tray is gone completly crazy. 

Now after clean boot Klipper icon is missing (earlier only sound icon was missing), that half icon which @Mark is talking about is actually Device notifier icon and it goes away if you open hidden item by clicking on the arrow.

When system boots up, if I do kquitapp plasmashell and then plasmashell, systray starts without these bugs.

Anyway, I cannot beleive that systray is still broken and we are at plasma 5.5.3 version...

Look at attached screenshots.
Comment 59 Nikola Schnelle 2016-01-14 22:55:26 UTC
Created attachment 96648 [details]
clean boot
Comment 60 Nikola Schnelle 2016-01-14 22:56:13 UTC
Created attachment 96649 [details]
after clicking on systray arrow
Comment 61 Joel Koreth 2016-01-15 03:23:32 UTC
I can confirm with Nikola Schnelle's behavior with recent 5.5.3 updates which forced me to go back to 5.4.3 causing the bug to be back to the sound applet disappearing. For context i had 5.5.3 on opensuse through the KF5 + KDE repo's.
Comment 62 Joel Koreth 2016-01-15 03:23:51 UTC
I can confirm with Nikola Schnelle's behavior with recent 5.5.3 updates which forced me to go back to 5.4.3 causing the bug to be back to the sound applet disappearing. For context i had 5.5.3 on opensuse through the KF5 + KDE repo's.
Comment 63 sedrubal 2016-01-21 18:35:55 UTC
*** Bug 358319 has been marked as a duplicate of this bug. ***
Comment 64 Robby Engelmann 2016-01-21 19:49:19 UTC
I can confirm Nikola Schnelle's screenshots of the tray. It also occurs in openSUSE Tumbleweed with Frameworks 5.18 and Plasma 5.5.3. Klipper is gone, hover effect over the empty space after startup shows device notifier after clicking on the empty space there is a large text behind the symbols of the tray.
Comment 65 Nikola Schnelle 2016-01-30 08:26:55 UTC
Can we have workaround implemented for this bug ("adding a shoddy timer delay before loading the applets" as David said) until fix in available in qt 5.6?
Note that bug https://bugreports.qt.io/browse/QTBUG-48870 is still not fixed in qt ("in progress"). Also qt 5.6 release is delayed again and again.

We have broken systray for months and we will have it broken in next months in stable product...
Comment 66 Marco Martin 2016-02-03 14:59:05 UTC
Git commit 13ad256cc40a657b54a236eef5c491fdf6b48ffc by Marco Martin.
Committed on 03/02/2016 at 14:58.
Pushed by mart into branch 'master'.

reset the model on list always shown/hide change

something really wrong is going on on the proxymodel updates
the wrong item gets removed from the list.
it may be  a wrong mapping between source and dest model
(doesn't seem so)
or may have been some misguided attempt by QML to recycle delegates

anyways resetting the model in some conditions even if expensive
seems to be the only way to workaround this.
Anyways this systray implementation is beyond any repair and
the rewritten version won't have to rely on so many models
and proxymodels
Related: bug 357627

M  +10   -3    applets/systemtray/plugin/tasksproxymodel.cpp
M  +4    -1    applets/systemtray/plugin/tasksproxymodel.h

http://commits.kde.org/plasma-workspace/13ad256cc40a657b54a236eef5c491fdf6b48ffc
Comment 67 Marco Martin 2016-02-04 10:56:48 UTC
Git commit 42a3d8accd4e494d343954ddaa916a6c618d94f3 by Marco Martin.
Committed on 04/02/2016 at 10:56.
Pushed by mart into branch 'Plasma/5.5'.

reset the model on list always shown/hide change

something really wrong is going on on the proxymodel updates
the wrong item gets removed from the list.
it may be  a wrong mapping between source and dest model
(doesn't seem so)
or may have been some misguided attempt by QML to recycle delegates

anyways resetting the model in some conditions even if expensive
seems to be the only way to workaround this.
Anyways this systray implementation is beyond any repair and
the rewritten version won't have to rely on so many models
and proxymodels
Related: bug 357627

M  +10   -3    applets/systemtray/plugin/tasksproxymodel.cpp
M  +4    -1    applets/systemtray/plugin/tasksproxymodel.h

http://commits.kde.org/plasma-workspace/42a3d8accd4e494d343954ddaa916a6c618d94f3
Comment 68 Fahad Alduraibi 2016-02-06 21:04:43 UTC
I have the same issue with audio icon disappearing, so now I went back to using kmix. However, the annoying problem is with a mysterious hidden icon which I couldn't find its originator. 

If I hide that icon from the System Tray entries it reappears after restarting the system.  So I checked the file "plasma-org.kde.plasma.desktop-appletsrc" to see what is happening and I find that it gets a new number every time:

hiddenItems=67108952,65011800

what are these numbers? and how can I find the app/service responsible for this entry?

(see the attached image)

Fedora 23, KDE 5.18 Qt 5.5.1
Comment 69 Fahad Alduraibi 2016-02-06 21:05:49 UTC
Created attachment 97062 [details]
hidden icon and entry
Comment 70 Anthony Fieroni 2016-02-08 07:34:18 UTC
This patch is a not proper fix, can you provide another or any ideas to resolve the issues within Qt
Comment 71 Anthony Fieroni 2016-02-08 07:36:24 UTC
*** Bug 357803 has been marked as a duplicate of this bug. ***
Comment 72 TrevizeDaneel 2016-02-13 12:29:27 UTC
I confirm the bug not been fixed yet.
Comment 73 David Edmundson 2016-02-25 01:09:35 UTC
When you say "it's not fixed yet" are you including this patch or not?

https://codereview.qt-project.org/#/c/141620/

This is the one that we've said is the actual relevant fix not just attempted workarounds.
Comment 74 Weng Xuetian 2016-02-25 17:51:21 UTC
(In reply to David Edmundson from comment #73)
> When you say "it's not fixed yet" are you including this patch or not?
> 
> https://codereview.qt-project.org/#/c/141620/
> 
> This is the one that we've said is the actual relevant fix not just
> attempted workarounds.

This is not the only bug in list view unfortunately. I didn't report it to qt because I have no stable way to reproduce it. See my comment #30.

I wonder replace all {begin,end}InsertRow with {begin,end}ResetModel in TaskListModel would workaround it.
Comment 75 Rex Dieter 2016-02-25 20:40:14 UTC
The patch referenced in comment #73 (applied against qtbase-5.5.1) seems to help, at least did so the first time I started plasmashell.  On my second login, plasma-pa was missing again.
Comment 76 Nikola Schnelle 2016-02-27 18:50:40 UTC
It seems that this is not fixed in qt 5.6 (users on plasma irc channel confirmed this), so can you please make this top priority for fixing?

Thank you.
Comment 77 Sudhir Khanger 2016-02-29 02:47:54 UTC
Created attachment 97603 [details]
gaps in system tray

As you can see in the screenshot patch is a fail for me too.

$ rpm -qa | grep qt5-qtbase
qt5-qtbase-5.5.1-12.fc23.x86_64

https://bodhi.fedoraproject.org/updates/FEDORA-2016-df9f4a6560
Comment 78 Weng Xuetian 2016-02-29 22:20:13 UTC
Welcome to test this patch to see if it can workaround this issue finally.

https://git.reviewboard.kde.org/r/127229/
Comment 79 Weng Xuetian 2016-03-01 16:23:34 UTC
Git commit 6a26adb50c0bfce031df55bf200ebc7f44420301 by Weng Xuetian.
Committed on 01/03/2016 at 16:05.
Pushed by xuetianweng into branch 'master'.

Workaround by always reset model when insert and remove from model.

M  +6    -5    applets/systemtray/plugin/tasklistmodel.cpp

http://commits.kde.org/plasma-workspace/6a26adb50c0bfce031df55bf200ebc7f44420301
Comment 80 Marco Martin 2016-03-01 16:35:25 UTC
Git commit 16584dc70ef9121876ded1604daf32774d1862e2 by Marco Martin, on behalf of Weng Xuetian.
Committed on 01/03/2016 at 16:34.
Pushed by mart into branch 'Plasma/5.5'.

Workaround by always reset model when insert and remove from model.

M  +6    -5    applets/systemtray/plugin/tasklistmodel.cpp

http://commits.kde.org/plasma-workspace/16584dc70ef9121876ded1604daf32774d1862e2
Comment 81 Raghavendra kamath 2016-03-01 19:16:34 UTC
After the Plasma 5.5.5 march bug-fix update, I don't get the systray arrow to show hidden icons. To make the arrow appear I insert a usb drive to trigger the device list icon. Dropbox icon is missing too.

I am using arch linux
Comment 82 Geoffrey Bonneville 2016-03-01 21:22:34 UTC
(In reply to Raghavendra kamath from comment #81)
> After the Plasma 5.5.5 march bug-fix update, I don't get the systray arrow
> to show hidden icons. To make the arrow appear I insert a usb drive to
> trigger the device list icon. Dropbox icon is missing too.
> 
> I am using arch linux

Same bug (Arch too). It's worst than before the patch :/
Comment 83 Mark 2016-03-02 02:50:02 UTC
(In reply to Raghavendra kamath from comment #81)
> Dropbox icon is missing too.

there is a bug in Dropbox 3.14.5 (and newer) that prevents the icon loading. Nothing to do with Plasma.
You should be able to get the icon back with this command

dropbox stop && dbus-launch dropbox start
Comment 85 Raghavendra kamath 2016-03-02 03:57:28 UTC
(In reply to Mark from comment #83)
> (In reply to Raghavendra kamath from comment #81)
> > Dropbox icon is missing too.
> 
> there is a bug in Dropbox 3.14.5 (and newer) that prevents the icon loading.
> Nothing to do with Plasma.
> You should be able to get the icon back with this command
> 
> dropbox stop && dbus-launch dropbox start

My main problem is that there is no arrow to show the hidden icons, I know dropbox has problems and it's not a major problem for me.
Just to inform you that command did nothing , the dropbox icon doesn't come back, again I am stating it's not the main issue here.

Because of the missing arrow we can't access notifications printers devices and other icons which are hidden.
Comment 86 Anthony Fieroni 2016-03-02 04:44:37 UTC
(In reply to Raghavendra kamath from comment #81)
> After the Plasma 5.5.5 march bug-fix update, I don't get the systray arrow
> to show hidden icons. To make the arrow appear I insert a usb drive to
> trigger the device list icon. Dropbox icon is missing too.
> 
> I am using arch linux

Comment 84 for arrow patch
Let's make more clarifications, release date is simple a date, all that present in stable braches, i.e. Plasma/5.5, is for daily use and stable. 
source=('git://anongit.kde.org/plasma-workspace.git#branch=Plasma/5.5')
Comment 87 Anthony Fieroni 2016-03-02 04:48:19 UTC
*** Bug 357803 has been marked as a duplicate of this bug. ***
Comment 88 Marco Martin 2016-03-02 10:58:58 UTC
*** Bug 359711 has been marked as a duplicate of this bug. ***
Comment 89 Victor B. Gonzalez 2016-03-04 13:25:25 UTC
Every time I get an update in Fedora 23 (updates-testing), I test out the system tray. First thing I look for is the missing audio icon, hidden icons, then the system tray configuration itself.

So far in the latest update (I saw what it was KDE 5.5, I forgot the rest, I'll explain) the system tray is finally working great. No missing audio icon after restarting plasmashell 3 times, I can get to the hidden system tray icons and go into the settings all without any problem.

This problem seems like it might finally be fixed once and for all?

The reason I forgot is because kinfo only launched once for me without a problem. Coming here to say how great things are caused me to open it again and it won't open anymore :(
Comment 90 Mark 2016-03-04 14:28:51 UTC
yes, things seem to be working well for me also with the update to plasma 5.5.5 on openSUSE Leap 42.1
Comment 91 Marco Martin 2016-03-09 11:57:16 UTC
Systemtray implementation is changed, this should be one of the problems fixed.
the empty places problems has definitely been solved
Comment 92 David Rosca 2016-04-02 10:07:53 UTC
*** Bug 361126 has been marked as a duplicate of this bug. ***
Comment 93 Matt Scheirer 2016-04-02 15:05:42 UTC
I waited a week, but it seems that the combination of Frameworks 5.20, Plasma 5.6, and Qt 5.6 on Arch has fixed the missing tray icon issue entirely. It is no longer occurring on three systems I've been testing for it all week, so if you are still suffering this bug try upgrading your system.
Comment 94 EMR_Kde 2016-08-02 11:36:35 UTC
Whatever was fixed by the sni proxy, once it was incorporated into the latest plasma update, it broke again... this time worse than before... nothing is showing up in my system tray other than native KDE apps.
At least before Pidgin/Java apps (though broken cause no context menu would come up) would at least SHOW UP there, now nothing.
Comment 95 Hrtmt Brng 2016-09-11 16:17:52 UTC
Created attachment 101035 [details]
missing systray icons

I have also missing systray icons. It happens on two computers with Debian testing (plasma 4.6.5, frameworks 5.25.0). On the second computer it seemed to happen after installing hplip. See screenshot. There should be network manager, volume control and bluetooth. I can neither see the icons nor click on them. I can not even connect to wifi. On the other computer hplip is not installed. But I have only volume control and a huge gap.
Comment 96 Christian Boltz 2016-09-11 19:51:04 UTC
This bug was closed as fixed (by a different implementation, therefore the bug status is "resolved unmaintained") in comment 91.

If you still see problems, please open a new bugreport. (Feel free to mention that it looks similar to this bugreport or something like that.)

General note: It's very likely that comments to a closed bug will be overlooked, especially if the closed bug already has nearly 100 comments.