| Summary: | Kickoff immediately selects entry under mouse in search | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Saeid Akbari <saeidscorp> |
| Component: | Application Launcher (Kickoff) widget | Assignee: | David Edmundson <kde> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | bugseforuns, hein, nate, notuxius, plasma-bugs |
| Priority: | NOR | Keywords: | usability |
| Version: | 5.12.5 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| See Also: | https://bugs.kde.org/show_bug.cgi?id=404655 | ||
| Latest Commit: | https://commits.kde.org/plasma-desktop/48b32259e73f374a456ed836ba44e3effe2baee5 | Version Fixed In: | |
| Sentry Crash Report: | |||
|
Description
Saeid Akbari
2018-08-21 07:29:58 UTC
My wife actually hit this yesterday and was very confused. She tried the same thing three times and got frustrated before I figured out what was happening and showed her a workaround. Perhaps we should require movement before selecting the item that's under the pointer, if it happens to be positioned where results will show up. Noticed that searchable entries in Kicker aren't highlighted if cursor was not moved Operating System: KDE neon Developer Edition KDE Plasma Version: 5.14.80 Qt Version: 5.11.1 KDE Frameworks Version: 5.51.0 Kernel Version: 4.15.0-34-generic Yup, The issue is resolved after a world update. OS: Gentoo Plasma Workspace: 5.12.5 (same as before) Qt: 5.9.6 KDE Framework: 5.46.0 If anyone still facing it after the update, it'll be reopened. Definitely not fixed, sadly. Still a problem in all versions of Plasma. Nate Graham, did you update KDE Frameworks? I think that fixed the issue for me as I only updated that and still using the same version of Plasma desktop. I'm a KDE developer and I run everything from git master. :) Patch under review: https://phabricator.kde.org/D15855 Git commit 48b32259e73f374a456ed836ba44e3effe2baee5 by Eike Hein. Committed on 24/10/2018 at 08:01. Pushed by hein into branch 'master'. Port Kickoff to shared view component using single-MouseArea pattern Summary: This: * Moves MouseArea out of the item delegate and uses a single MouseArea per view. * Ports all the pages to using a shared view component. Aside from saving a lot of QObjects, using a single MouseArea also has the advantage that MouseArea.positionChanged isn't fired when QQuickWindow synthesizes a hover event per frame, unlike MouseArea .containsMouse. This fixes a user-reported bug in the search page, and the same pattern is used by various other UIs we have for similar reasons. Reviewers: #plasma, ngraham, davidedmundson Reviewed By: #plasma, davidedmundson Subscribers: davidedmundson, ngraham, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D15855 M +42 -52 applets/kickoff/package/contents/ui/ApplicationsView.qml M +21 -34 applets/kickoff/package/contents/ui/BaseView.qml M +1 -1 applets/kickoff/package/contents/ui/Breadcrumb.qml M +39 -56 applets/kickoff/package/contents/ui/FavoritesView.qml M +66 -135 applets/kickoff/package/contents/ui/KickoffItem.qml A +170 -0 applets/kickoff/package/contents/ui/KickoffListView.qml [License: GPL (v2+)] M +23 -33 applets/kickoff/package/contents/ui/SearchView.qml https://commits.kde.org/plasma-desktop/48b32259e73f374a456ed836ba44e3effe2baee5 |