Summary: | Application launcher switches selection on autocomplete to mouse position | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Matija <mata987> |
Component: | Application Launcher (Kickoff) widget | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | bugseforuns, jat255, mikel5764, nate, noahadvs, philippe.roubach, putr4.s, qydwhotmail, rohan, shubhamarora |
Priority: | HI | Keywords: | usability |
Version: | 5.27.0 | ||
Target Milestone: | 1.0 | ||
Platform: | Neon | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/-/commit/717465388631796747cd8f2061650a0abed19bee | Version Fixed In: | 5.27.7 |
Sentry Crash Report: | |||
Attachments: |
Screen recording of reproducing the bug
Screen recording of an item getting selected despite the mouse not being over the launcher Video for debugging idea from Comment #5 |
Description
Matija
2023-02-21 12:54:58 UTC
I can reproduce this on Plasma 5.27.4 on Wayland, although the reproduction steps are a bit different in my case (will attach a screen recording showing the issue as well): 1. Open the application launcher with the Meta/Super key 2. Press the E key 3. Move the cursor to the 4th item, making sure it is highlighted (and not touch the mouse again from here on out) 4. Close the application launcher by pressing Esc 5. Open the application launcher with the Meta/Super key again 6. Press the E key 7. Wait a second or two 8. Press the D key By step 8 the issue appears for me, but if I stop at step 6 the issue does not appear, and if I press E followed by D quickly the issue also doesn't appear, but if I type "ed" quickly, wait a second and then type another letter the issue appears again. Also, the issue has been affecting me for a while, but only now have I found out good steps to reproduce it thanks to this bug report. Created attachment 158066 [details]
Screen recording of reproducing the bug
Created attachment 158067 [details]
Screen recording of an item getting selected despite the mouse not being over the launcher
In addition, once the bug has been triggered plasmashell still seems to remember the old cursor position even after closing the launcher and having the mouse be nowhere near it at all.
(In reply to Prajna Sariputra from comment #3) > Created attachment 158067 [details] > Screen recording of an item getting selected despite the mouse not being > over the launcher > > In addition, once the bug has been triggered plasmashell still seems to > remember the old cursor position even after closing the launcher and having > the mouse be nowhere near it at all. See bug 454349 (In reply to Prajna Sariputra from comment #1) > I can reproduce this on Plasma 5.27.4 on Wayland, although the reproduction > steps are a bit different in my case (will attach a screen recording showing > the issue as well): > > 1. Open the application launcher with the Meta/Super key > 2. Press the E key > 3. Move the cursor to the 4th item, making sure it is highlighted (and not > touch the mouse again from here on out) > 4. Close the application launcher by pressing Esc > 5. Open the application launcher with the Meta/Super key again > 6. Press the E key > 7. Wait a second or two > 8. Press the D key > > By step 8 the issue appears for me, but if I stop at step 6 the issue does > not appear, and if I press E followed by D quickly the issue also doesn't > appear, but if I type "ed" quickly, wait a second and then type another > letter the issue appears again. > > Also, the issue has been affecting me for a while, but only now have I found > out good steps to reproduce it thanks to this bug report. The highlighting problem has something to do with the accuracy of the search and memorizing/matching the position under cursor. A few tips for debugging: 1. Nothing happens when starting the search term with one letter. The case when it's triggered is when you start deleting chars from a word and end with one letter. 2. If I type really fast "man" or "file" it doesn't jump to mouse cursor. Why? I come from web dev world, so I presume because keyup/down event doesn't fire in that case. 3. Only if typed slower, when keyup/down events are fired, it seems to save the resulting list on each key event and then by some logic, saves the position of the item that does satisfy the search term (in one of the steps between additional characters), and passive mouse pointer activates it. If under the mouse cursor ever existed (in between each char entered) a positive find, it will jump to that instead of the first item. So: 1. open launcher 2. type char ("f" for instance) 3. nothing happens as there are too many results and this is only the first step in searching 4. wait a second 5. type another char ("i" for instance) 6. it jumps to the item under mouse cursor as there was a positive match in the step before (although not in current step anymore) vs 1. open launcher 2. type FAST "fi" 3. there are no akward jumps Attaching a video recording of the above Created attachment 158072 [details] Video for debugging idea from Comment #5 *** Bug 467637 has been marked as a duplicate of this bug. *** *** Bug 467304 has been marked as a duplicate of this bug. *** A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/1619 A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/1620 Git commit fc85def499318caae360462568e239a1eeb7d61e by Fushan Wen. Committed on 29/07/2023 at 02:11. Pushed by fusionfuture into branch 'master'. applets/kickoff: add blocker to prevent focus change in search view This adds a mouse position interceptor to record the mouse position after entering a new search term, and if there is no mouse movement, block focus on child items. FIXED-IN: 5.27.7 M +1 -1 applets/kickoff/package/contents/ui/AbstractKickoffItemDelegate.qml M +36 -3 applets/kickoff/package/contents/ui/FullRepresentation.qml https://invent.kde.org/plasma/plasma-desktop/-/commit/fc85def499318caae360462568e239a1eeb7d61e Git commit 717465388631796747cd8f2061650a0abed19bee by Fushan Wen. Committed on 29/07/2023 at 02:18. Pushed by fusionfuture into branch 'Plasma/5.27'. applets/kickoff: add blocker to prevent focus change in search view This adds a mouse position interceptor to record the mouse position after entering a new search term, and if there is no mouse movement, block focus on child items. FIXED-IN: 5.27.7 (cherry picked from commit fc85def499318caae360462568e239a1eeb7d61e) M +1 -1 applets/kickoff/package/contents/ui/AbstractKickoffItemDelegate.qml M +36 -3 applets/kickoff/package/contents/ui/FullRepresentation.qml https://invent.kde.org/plasma/plasma-desktop/-/commit/717465388631796747cd8f2061650a0abed19bee |