| Summary: | When I hover over a widget in the Widget Explorer, the hovering effect is also briefly applied to another widget | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Patrick Silva <bugseforuns> |
| Component: | Widget Explorer | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | justin.zobel, kde, nate |
| Priority: | NOR | ||
| Version: | master | ||
| Target Milestone: | 1.0 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/commit/5298171862ded64cfd637d313bd1fa85ba66a36c | Version Fixed In: | 5.20.2 |
| Attachments: | screen recording | ||
Can confirm on KDE Neon Unstable git builds. I added
onCurrentIndexChanged: console.log(currentIndex);
I changes in a way that matches the video. i.e it randomly prints the index of the top right item for a brief moment.
WTF. In the delegate it has this line:
onExited: delegate.GridView.view.currentIndex = index - 1
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/204 Git commit 7cdf189f5868c2ea56a6a461d49909deb0e76504 by Nate Graham, on behalf of David Edmundson. Committed on 26/10/2020 at 00:22. Pushed by ngraham into branch 'master'. [Widget Explorer] Don't set the list index to something random on mouse exit -1 is an invalid index which makes sense. We know mouse exits get processes before any other item's mouse in. So this would hide the highlight. The previous code doesn't make sense M +1 -1 desktoppackage/contents/explorer/AppletDelegate.qml https://invent.kde.org/plasma/plasma-desktop/commit/7cdf189f5868c2ea56a6a461d49909deb0e76504 Git commit 5298171862ded64cfd637d313bd1fa85ba66a36c by Nate Graham, on behalf of David Edmundson. Committed on 26/10/2020 at 00:30. Pushed by ngraham into branch 'Plasma/5.20'. [Widget Explorer] Don't set the list index to something random on mouse exit -1 is an invalid index which makes sense. We know mouse exits get processes before any other item's mouse in. So this would hide the highlight. The previous code doesn't make sense (cherry picked from commit 7cdf189f5868c2ea56a6a461d49909deb0e76504) M +1 -1 desktoppackage/contents/explorer/AppletDelegate.qml https://invent.kde.org/plasma/plasma-desktop/commit/5298171862ded64cfd637d313bd1fa85ba66a36c |
Created attachment 132682 [details] screen recording SUMMARY Watch the attached screen recording and observe "Activity Pager" and "Application Dashboard" widgets while I hover over another one. EXPECTED RESULT hovering effect should only be applied to an hovered widget SOFTWARE/OS VERSIONS Operating System: KDE neon Unstable Edition KDE Plasma Version: 5.20.80 KDE Frameworks Version: 5.76.0 Qt Version: 5.15.0 ADDITIONAL INFORMATION I can also reproduce this behavior on Arch Linux running Plasma 5.20.1.