Bug 428174 - When I hover over a widget in the Widget Explorer, the hovering effect is also briefly applied to another widget
Summary: When I hover over a widget in the Widget Explorer, the hovering effect is als...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Widget Explorer (show other bugs)
Version: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-10-24 12:15 UTC by Patrick Silva
Modified: 2020-10-26 00:30 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.20.2


Attachments
screen recording (1.20 MB, video/webm)
2020-10-24 12:15 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2020-10-24 12:15:44 UTC
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.
Comment 1 Justin Zobel 2020-10-25 09:34:44 UTC
Can confirm on KDE Neon Unstable git builds.
Comment 2 David Edmundson 2020-10-25 23:45:07 UTC
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.
Comment 3 David Edmundson 2020-10-25 23:46:10 UTC
WTF. In the delegate it has this line:


            onExited: delegate.GridView.view.currentIndex = index - 1
Comment 4 Bug Janitor Service 2020-10-25 23:49:06 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/204
Comment 5 Nate Graham 2020-10-26 00:29:47 UTC
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
Comment 6 Nate Graham 2020-10-26 00:30:15 UTC
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