Bug 443076

Summary:	item under mouse pointer is launched instead of the first search result
Product:	[Plasma] plasmashell	Reporter:	Mariusz Libera <mariusz.libera>
Component:	Application Launcher (Kickoff) widget	Assignee:	David Edmundson <kde>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugseforuns, mikel5764, nate, noahadvs, plasma-bugs-null
Priority:	VHI	Keywords:	regression, usability
Version First Reported In:	5.25.4
Target Milestone:	1.0
Platform:	Arch Linux
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-desktop/commit/7a4dd40f6bea0434f6fd7842897f28843e7e05e2	Version Fixed/Implemented In:	5.23
Sentry Crash Report:

Description Mariusz Libera 2021-09-28 15:06:51 UTC

STEPS TO REPRODUCE
1. Position the mouse pointer where the launcher window will pop up.
2. Don't move the mouse.
3. Press [meta] to open the launcher.
4. Type something to search.
5. Press [enter] to launch the first result on the list.

OBSERVED RESULT
Item under the mouse pointer is highlighted and launched.

EXPECTED RESULT
First result of the search is highlighted and launched.

ADDITIONAL INFORMATION
This used to work correctly.

Comment 1 Patrick Silva 2021-09-28 18:15:43 UTC

I'm unable to reproduce with kickoff included in Plasma.

Plasma 5.23 beta, Arch Linux.

Comment 2 Nate Graham 2021-09-28 20:34:56 UTC

I can reproduce the issue. This was working in the past, but I guess it got broken in one of many many code refactors.

Comment 3 Noah Davis 2021-10-04 02:36:02 UTC

Will be fixed by https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/573

Comment 4 Noah Davis 2021-10-06 17:07:14 UTC

Git commit 9d48efbb11bdcabdfd7b459c721f55e6259430d3 by Noah Davis.
Committed on 06/10/2021 at 16:43.
Pushed by ndavis into branch 'master'.

kickoff: fix hover changing index when moving between search and normal pages

M  +3    -0    applets/kickoff/package/contents/ui/KickoffItemDelegate.qml

https://invent.kde.org/plasma/plasma-desktop/commit/9d48efbb11bdcabdfd7b459c721f55e6259430d3

Comment 5 Noah Davis 2021-10-06 17:11:16 UTC

Git commit 7a4dd40f6bea0434f6fd7842897f28843e7e05e2 by Noah Davis.
Committed on 06/10/2021 at 17:10.
Pushed by ndavis into branch 'Plasma/5.23'.

kickoff: fix hover changing index when moving between search and normal pages
(cherry picked from commit 9d48efbb11bdcabdfd7b459c721f55e6259430d3)

M  +3    -0    applets/kickoff/package/contents/ui/KickoffItemDelegate.qml

https://invent.kde.org/plasma/plasma-desktop/commit/7a4dd40f6bea0434f6fd7842897f28843e7e05e2

Comment 6 Mariusz Libera 2022-08-20 14:37:16 UTC

This is broken again.
5.25.4

Comment 7 Nate Graham 2022-08-20 22:38:18 UTC

Yeah. Thankfully it's already fixed again in Plasma 5.25.5 and 5.26. :)