https://bugs.kde.org/show_bug.cgi?id=501350
cwo <cwo....@posteo.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |https://invent.kde.org/plas
| |ma/milou/-/commit/ed5257558
| |eeb994071be14a9c799d4a34e11
| |e169
Version Fixed In| |6.4.0
Resolution|--- |FIXED
Status|ASSIGNED |RESOLVED
--- Comment #3 from cwo <cwo....@posteo.net> ---
Git commit ed5257558eeb994071be14a9c799d4a34e11e169 by Christoph Wolk.
Committed on 12/03/2025 at 18:45.
Pushed by cwo into branch 'master'.
lib: simplify mouse hover handling
ResultsView/ResultDelegate do some complicated mouse interaction
handling: the delegate listens to a HoverHandler's pointChanged signal.
which updates in a lot of circumstances, and then filters based on
keeping track whether the mouse moved to not activate with keyboard-only
use. This doesn't work well if there is mixed keyboard/mouse use, where
arrow key navigation becomes useless almost as soon as the mouse is
involved. And it isn't really necessary in the first place, as the real
problem is connecting to such a busy signal.
Instead, we drop the whole state-tracking jazz and even the
HoverHandler, and just listen to the delegate's own hovered property.
hovered only updates after the mouse moves, so we get essentially
the same thing for free but without the edge case bugs.
FIXED-IN: 6.4.0
M +1 -6 lib/mousehelper.cpp
M +0 -2 lib/mousehelper.h
M +6 -13 lib/qml/ResultDelegate.qml
M +0 -9 lib/qml/ResultsView.qml
https://invent.kde.org/plasma/milou/-/commit/ed5257558eeb994071be14a9c799d4a34e11e169
--
You are receiving this mail because:
You are watching all bug changes.
