Bug 482005 - Can't launch app with enter key, works fine with mouse
Summary: Can't launch app with enter key, works fine with mouse
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Application Menu (Kicker) (show other bugs)
Version: master
Platform: Arch Linux Linux
: HI normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: qt6
: 482882 (view as bug list)
Depends on:
Blocks:
 
Reported: 2024-02-29 10:08 UTC by n0v0x
Modified: 2024-03-08 19:17 UTC (History)
12 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.1


Attachments
dashboard (1.51 MB, image/png)
2024-03-02 16:46 UTC, duha.bugs
Details

Note You need to log in before you can comment on or make changes to this bug.
Description n0v0x 2024-02-29 10:08:37 UTC
STEPS TO REPRODUCE
1. Switch to Application Menu (Kicker)
2. Type beginning of app name, navigate to wanted app with arrows, validate with enter

OBSERVED RESULT
App doesn't launch (will launch if selected with mouse)


EXPECTED RESULT
App launch


SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Arch Linux x86_64
KDE Plasma Version: 6.0.0 
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2

ADDITIONAL INFORMATION
Comment 1 David Edmundson 2024-02-29 14:39:37 UTC
I can reproduce, but it's more complicated than reported. It sometimes works for search results, it doesn't work navigating in submenus.
Comment 2 duha.bugs 2024-03-02 15:12:33 UTC
The bug appears both in wayland and X11 on Plasma 6.

Does work correctly in Plasma 5.

Findings:
If you search for something and don't use your arrow keys it opens the (top most) App as expected. Because the search field has focus.

If you search for something and use the arrow keys  -> The search field has no focus anymore
Press enter and the search field gets focus.
Press enter and it will always open the top most App

If you navigate with the arrow keys with out searching:
After pressing enter the search field gets foucs.
Nothing gets opened because there is no top most app.
Comment 3 duha.bugs 2024-03-02 16:46:57 UTC
Created attachment 166310 [details]
dashboard

Dashboard search is also broken ( I think this shares a codebase?)

Can't use enter to open applications, but allows multiline (should not). Afaik this worked correctly at one point in plasma 6. Not sure if this is a QT or KDE bug.

KDE Plasma Version: 6.0.80
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2
Kernel Version: 6.7.5-arch1-1 (64-bit)
Graphics Platform: Wayland
Comment 4 paninomaninodesu 2024-03-04 00:36:24 UTC
I use "Application Menu" (that simpler one).
Hiting enter doesn't open the program.
Other thing I noticed, when using the arrows to enter a category no programas are selected. Shouldn't the first program on a category be selected automatically?

Tested only on X11.
Comment 5 Bug Janitor Service 2024-03-04 15:34:52 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2080
Comment 6 Marco Martin 2024-03-05 11:18:14 UTC
Git commit 087491446de290b4c1acd6d0318986307309f32b by Marco Martin.
Committed on 04/03/2024 at 16:46.
Pushed by mart into branch 'master'.

Disable internal Key navigation

Kicker has completely custom key navigation, from the code it used to
have key navigation managed internally by ListView but there was a stray
keyNavigationWraps there , which in Qt6 makes it enable key navigation
anyways
Explicitly disable it and get back to existing code managing the keys.
also make sure the current item has focus, so enter to start works again

M  +13   -10   applets/kicker/package/contents/ui/ItemListView.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/087491446de290b4c1acd6d0318986307309f32b
Comment 7 Marco Martin 2024-03-05 11:22:11 UTC
Git commit 4a6ef6061104e31fcc6ddb9c4e6f970856d2fe89 by Marco Martin.
Committed on 05/03/2024 at 11:20.
Pushed by mart into branch 'Plasma/6.0'.

Disable internal Key navigation

Kicker has completely custom key navigation, from the code it used to
have key navigation managed internally by ListView but there was a stray
keyNavigationWraps there , which in Qt6 makes it enable key navigation
anyways
Explicitly disable it and get back to existing code managing the keys.
also make sure the current item has focus, so enter to start works again

M  +13   -10   applets/kicker/package/contents/ui/ItemListView.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/4a6ef6061104e31fcc6ddb9c4e6f970856d2fe89
Comment 8 Patrick Silva 2024-03-08 19:17:27 UTC
*** Bug 482882 has been marked as a duplicate of this bug. ***