Users have reported a number of bugs against TM tooltips, e.g. initially
coming up with crazy sizes, or showing window duplicates. This all comes
down to a race conditions causing stale data and ugly code that's using
visible props to swap between two alternate UI implementations instead
of doing clean conditional loading.
With this cleanup patch the problems should be gone, as well as various
ReferenceErrors spewed onto the tty.
BUG:382571
BUG:385947
BUG:389469
BUG:388749
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
To be honest, I didn't fully figure it out. But when the assignment in Tasks.qml was an integer to the delegate, it meant that the DelegateModel in the delegate sometimes started out with an undefined rootIndex and therefore bad data - almost as if the binding never got evaluated. Always assigning a freshly-made model index directly in the mouse event handler fixed it, and I prefer it style-wise anyway.
Fix crash on Qt <5.11
This makes no sense, but fixes the crash reliably in testing. Since it
works on 5.11, this may be a bug that got fixed in Qt. I didn't bisect
it though.
I have tested the modified patch on the same environment with Qt 5.10.1 where I found the crash with the previous version in 5.13.1 tars.
plasmashell now seems to start without issue on login