Bug 454349 - Mouse hover selection in search result persists across kickoff openings
Summary: Mouse hover selection in search result persists across kickoff openings
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Unclassified
Component: Application Launcher (Kickoff) (show other bugs)
Version: 5.24.90
Platform: Archlinux Packages Linux
: HI normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
: 456991 457092 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-05-24 17:42 UTC by Matej Mrenica
Modified: 2022-08-03 03:14 UTC (History)
11 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.25.5


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Matej Mrenica 2022-05-24 17:42:38 UTC
STEPS TO REPRODUCE
1. Open Kicker
2. Type something

OBSERVED RESULT
There will be results and the second one will be highlighted, also if you press the arrow up key to select the first item
it would sometimes move back to the second item on it own.

EXPECTED RESULT
The first one is highlighted

SOFTWARE/OS VERSIONS
Plasma 5.25 Beta, Arch Linux, Wayland
Comment 1 Felipe Kinoshita 2022-05-24 19:20:38 UTC
Cannot reproduce
Comment 2 Joshua T 2022-05-24 22:04:58 UTC
I have experienced this on Arch with Plasma 5.24.90 beta. It seemed to go away after an update. Try updating with "sudo pacman -Syuu". I also disabled the "Desktop Search" application launcher plugin, so that may have fixed the issue.
Comment 3 Nate Graham 2022-05-25 16:25:35 UTC
Are you sure you're using Kicker and not Kickoff?
Comment 4 Matej Mrenica 2022-05-25 16:48:14 UTC
I did mean Kickoff , sorry.
Comment 5 Nate Graham 2022-07-05 17:06:06 UTC
Is it possible that the cursor is above the place where the second item appears?
Comment 6 Matej Mrenica 2022-07-05 17:19:11 UTC
(In reply to Nate Graham from comment #5)
> Is it possible that the cursor is above the place where the second item
> appears?

No, my cursor is on the other side of the screen. Also, currently, it's not the second item it's the fifth. Does no one else have this issue?
Comment 7 Nate Graham 2022-07-06 15:12:51 UTC
Seems random-ish. I just managed to reproduce it, then I closed kickoff and tried again and it worked properly.
Comment 8 Johan Sköld 2022-07-18 07:16:05 UTC
I can reproduce this consistently as well in kickoff, but I also seem to have figured out what happens (but not why).

1. Open kickoff and search for something
2. Highlight one of the options with the mouse
3. Press escape to close kickoff
4. Move the cursor away
5. Open kickoff and search for something again (it doesn't have to be the same term)

When I do that, it selects the same entry that I hovered with the mouse in step 2. So if I hovered the third entry, it will select the third entry.

It sometimes even reproduces without searching for anything once you re-open kickoff, if you had the cursor in a position where it would highlight something in the default view. If I search for something and hover the 6th entry on the left side of the view, once I re-open kickoff it selects "Games" without me doing anything.

KDE Plasma Version: 5.25.3
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5
Kernel Version: 5.18.12-arch1-1 (64-bit)
Graphics Platform: Wayland
Comment 9 Patrick Silva 2022-07-18 11:30:50 UTC
(In reply to Johan Sköld from comment #8)
> 1. Open kickoff and search for something
> 2. Highlight one of the options with the mouse
> 3. Press escape to close kickoff
> 4. Move the cursor away
> 5. Open kickoff and search for something again (it doesn't have to be the
> same term)
> 
> When I do that, it selects the same entry that I hovered with the mouse in
> step 2. So if I hovered the third entry, it will select the third entry.
>

Can reproduce on neon unstable.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.25.80
KDE Frameworks Version: 5.97.0
Qt Version: 5.15.5
Graphics Platform: Wayland
Comment 10 Nate Graham 2022-07-19 17:03:32 UTC
I can reproduce that 100%!
Comment 11 Nate Graham 2022-07-22 18:20:09 UTC
*** Bug 456991 has been marked as a duplicate of this bug. ***
Comment 12 Nate Graham 2022-07-24 20:55:54 UTC
*** Bug 457092 has been marked as a duplicate of this bug. ***
Comment 13 Nate Graham 2022-08-02 22:48:58 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/1074
Comment 14 Nate Graham 2022-08-03 02:55:23 UTC
Git commit cfb520e3b39714edb6e9d406bc0e1c19667f0281 by Nate Graham.
Committed on 03/08/2022 at 02:54.
Pushed by ngraham into branch 'master'.

Revert "Use onEntered in KickoffItemDelegate"

This reverts commit 8042ae81d97cf27be1da652272d2dc86bfbbf042.

This change slightly improved performance, but subtle implementation
differences between the onEntered: and onPositionChanged: handlers in
MouseArea triggered two subtle and annoying bugs that users have
reported a bunch of duplicates of.

A way to fix the bugs using onEntered has not been found, so let's revert
the change for now; slightly worse performance is a less severe issue then
these bugs are.
Related: bug 455674
FIXED-IN: 5.25.5

M  +6    -9    applets/kickoff/package/contents/ui/KickoffItemDelegate.qml
M  +0    -11   applets/kickoff/package/contents/ui/KickoffListView.qml

https://invent.kde.org/plasma/plasma-desktop/commit/cfb520e3b39714edb6e9d406bc0e1c19667f0281
Comment 15 Nate Graham 2022-08-03 03:14:51 UTC
Git commit 3c87dc68746100960263ca35b400442170513474 by Nate Graham.
Committed on 03/08/2022 at 03:14.
Pushed by ngraham into branch 'Plasma/5.25'.

Revert "Use onEntered in KickoffItemDelegate"

This reverts commit 8042ae81d97cf27be1da652272d2dc86bfbbf042.

This change slightly improved performance, but subtle implementation
differences between the onEntered: and onPositionChanged: handlers in
MouseArea triggered two subtle and annoying bugs that users have
reported a bunch of duplicates of.

A way to fix the bugs using onEntered has not been found, so let's revert
the change for now; slightly worse performance is a less severe issue then
these bugs are.
Related: bug 455674, bug 456993
FIXED-IN: 5.25.5
(cherry picked from commit cfb520e3b39714edb6e9d406bc0e1c19667f0281)

M  +6    -9    applets/kickoff/package/contents/ui/KickoffItemDelegate.qml
M  +0    -11   applets/kickoff/package/contents/ui/KickoffListView.qml

https://invent.kde.org/plasma/plasma-desktop/commit/3c87dc68746100960263ca35b400442170513474