| Summary: | Kickoff inappropriately pre-selects the category under the cursor position when it opens | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Patrick Silva <bugseforuns> |
| Component: | Application Launcher (Kickoff) | Assignee: | David Edmundson <kde> |
| Status: | CONFIRMED --- | ||
| Severity: | minor | CC: | dev.bacteriostat, mikel5764, miranda, nate, plasma-bugs, qydwhotmail |
| Priority: | NOR | ||
| Version: | 5.26.4 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/commit/6f809058a6e0a4711bf23af118c091e683eea8bd | Version Fixed In: | |
| Sentry Crash Report: | |||
|
Description
Patrick Silva
2021-12-20 13:02:17 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1310 A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1311 Git commit 5d524ea52e73748582f8d1b8e9eb9d52e8bfccb0 by Fushan Wen. Committed on 26/12/2021 at 14:54. Pushed by fusionfuture into branch 'master'. applets/kicker: Use single-shot timer in TriangleMouseFilter The timer does not need to repeat sending hover events. M +1 -0 applets/kicker/plugin/trianglemousefilter.cpp https://invent.kde.org/plasma/plasma-workspace/commit/5d524ea52e73748582f8d1b8e9eb9d52e8bfccb0 A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1314 Git commit 6f809058a6e0a4711bf23af118c091e683eea8bd by David Edmundson. Committed on 02/01/2022 at 20:41. Pushed by davidedmundson into branch 'master'. [kicker] Cache position of hover enter events before replaying The triangle mouse blocks events that happen within a given region and conditions, then replays those events when those conditions get broken. If we intercept an enter event, we capture that under m_interceptedHoverItem so that if conditions change we can replay that hover enter event before we then start sending move events. Currently we replay that hover event using the current mouse position. This causes an issue for kickoff that has additional code to look for actual move events in order to avoid selecting the current item on scroll. Both codepaths are correct standalone, but together form a bug. This patch caches the relative position of a HoverEnter event. Should we replay the enter event, we do so with the position of the enter. We then can replay a HoverMove to get the up-to-date location. M +9 -1 applets/kicker/plugin/trianglemousefilter.cpp M +3 -2 applets/kicker/plugin/trianglemousefilter.h https://invent.kde.org/plasma/plasma-workspace/commit/6f809058a6e0a4711bf23af118c091e683eea8bd This bug is still reproducible on Plasma 5.24 beta. Operating System: Arch Linux KDE Plasma Version: 5.23.90 KDE Frameworks Version: 5.90.0 Qt Version: 5.15.2 Graphics Platform: Wayland Can repropduce if I hover over 'Lost & found' category in the step 2. Operating System: Arch Linux KDE Plasma Version: 5.24.90 KDE Frameworks Version: 5.94.0 Qt Version: 5.15.4 Graphics Platform: Wayland Can also reproduce on neon unstable by hovering over the last category in the step 2. *** This bug has been marked as a duplicate of bug 438950 *** it's still reproducible on neon unstable, but not consistently. Can reproduce sometimes, like Patrick. |