| Summary: | Icon pinned to task manager shows wide tooltip in just started plasma session | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Patrick Silva <bugseforuns> |
| Component: | Task Manager and Icons-Only Task Manager | Assignee: | Eike Hein <hein> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | kde, nate, petr.nehez, plasma-bugs, rootlexx, subdiff |
| Priority: | NOR | ||
| Version: | 5.13.0 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/plasma-desktop/1e218b405bee4d75a5d26a4c28da2724d967953a | Version Fixed In: | |
| Attachments: | wide tooltip | ||
*** This bug has been marked as a duplicate of bug 380432 *** this report is not a duplicate or 380432 is not fixed in plasma 5.10.4. The problem reported here happens on Arch running plasma 5.10.4. True, thanks for catching. Kai? I guess the icon's now right but the dialog sizing code wasn't adjusted. I've occasionally seen this happening with Plasma tooltips in general but I cannot reproduce it as consistently as this bug describes. (I don't know how up to date this system here is, so I might not have the patch here that I added in Bug 380432, need to check again) *** Bug 384600 has been marked as a duplicate of this bug. *** Potential fix: https://phabricator.kde.org/D7769 Maybe you can test it out Dr. Chapatin? (In reply to Roman Gilg from comment #7) > Potential fix: https://phabricator.kde.org/D7769 > > Maybe you can test it out Dr. Chapatin? Sorry, I have no technical knowledge to test patches. It's not happening anymore on Arch, plasma 5.11, frameworks 5.39. My mistake, just happened. It's not so easy reproduce it. *** Bug 385947 has been marked as a duplicate of this bug. *** reproducible in plasma 5.12 beta, Arch Linux. *** Bug 393717 has been marked as a duplicate of this bug. *** Reproducible on Arch Linux ruuning plasma 5.13 when I have two or more icons pinned to my plasma panel. Tooltip width increases according amount of pinned icons. That last bit of information was critical. I can reproduce now. Git commit 24803dc9dc4000e06edb60a3a8afe5925f0d72d7 by Eike Hein. Committed on 19/06/2018 at 11:22. Pushed by hein into branch 'Plasma/5.12'. Fix tooltip woes Summary: 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. Related: bug 385947, bug 389469, bug 388749 Reviewers: broulik, davidedmundson, romangg Subscribers: plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D13602 M +1 -1 applets/taskmanager/package/contents/ui/Task.qml M +27 -11 applets/taskmanager/package/contents/ui/ToolTipDelegate.qml M +1 -1 applets/taskmanager/package/contents/ui/ToolTipInstance.qml https://commits.kde.org/plasma-desktop/24803dc9dc4000e06edb60a3a8afe5925f0d72d7 Git commit 1ab6ae9a8a36068294c487896dd512c693332b08 by David Edmundson. Committed on 20/06/2018 at 07:16. Pushed by davidedmundson into branch 'Plasma/5.12'. Revert "Fix tooltip woes" This reverts commit 24803dc9dc4000e06edb60a3a8afe5925f0d72d7. Even though this commit is perfectly sensible it seems to introduce a crash on startup for Qt 5.10. Will revert for now, we can revisit later Related: bug 385947, bug 389469, bug 388749 M +1 -1 applets/taskmanager/package/contents/ui/Task.qml M +11 -27 applets/taskmanager/package/contents/ui/ToolTipDelegate.qml M +1 -1 applets/taskmanager/package/contents/ui/ToolTipInstance.qml https://commits.kde.org/plasma-desktop/1ab6ae9a8a36068294c487896dd512c693332b08 Git commit 1e218b405bee4d75a5d26a4c28da2724d967953a by Eike Hein. Committed on 22/06/2018 at 21:05. Pushed by hein into branch 'Plasma/5.12'. Fix tooltip woes Summary: 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. Related: bug 385947, bug 389469, bug 388749 Reviewers: broulik, davidedmundson, romangg Reviewed By: davidedmundson Subscribers: rikmills, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D13602 M +1 -1 applets/taskmanager/package/contents/ui/Task.qml M +33 -18 applets/taskmanager/package/contents/ui/ToolTipDelegate.qml M +1 -1 applets/taskmanager/package/contents/ui/ToolTipInstance.qml https://commits.kde.org/plasma-desktop/1e218b405bee4d75a5d26a4c28da2724d967953a |
Created attachment 106782 [details] wide tooltip pin any icon to your task manager restart your plasma session mouseover your pinned icon See my screenshot