https://bugs.kde.org/show_bug.cgi?id=525321

cwo <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/plasma-nm/-/commit/fbf2d |ma/plasma-nm/-/commit/eb8f4
                   |8181f8aa7ce96d7edac70384395 |ec47d7f726e52612ab440d7a39b
                   |cce71c08                    |9a785bff

--- Comment #5 from cwo <[email protected]> ---
Git commit eb8f4ec47d7f726e52612ab440d7a39b9a785bff by Christoph Wolk.
Committed on 06/10/2026 at 13:31.
Pushed by cwo into branch 'Plasma/6.6'.

applet: don't prevent moving focus from password to search field

6c46b8c2fc0017f5afa4e02340dbadb3730226c9 introduced a workaround for
ExpandableListItem moving focus on hover, which interrupted the user
while entering passwords if they accidentally moved the pointer. To
do this, it forced activeFocus on the password field if it loses
activeFocus while still having focus and if the new activeFocusItem's
focusReason is falsy (which it is when hovering over a ConnectionItem,
because it's an ExpandableListItem, which is not a Control and does not
have a focusReason, so it's undefined). This however breaks clicking the
search field, because MouseFocusReason is 0 and so falsy as well, while
focusing it with Ctrl+F still works as that sets ShortcutFocusReason.

Instead, explicitly check whether we have a focusReason (and if we do,
whether it's OtherFocusReason, which is what ListView sets when the
currentItem is changed if it were a control). This avoids the misleading
cast from enum to bool, and makes clicking the search field work again.
This way of doing it remains hacky, and gives awkward UX in edge cases
that are hard to fix, so we add a comment explaining this with a
reminder to replace this with a better implementation when ELI is
changed to allow this.
FIXED-IN: 6.7.6


(cherry picked from commit 2ba4a3ce55c32e432e9bf7c1fe248c56eb1b13de)

Co-authored-by: Christoph Wolk <[email protected]>

M  +8    -1    applet/ConnectionItem.qml

https://invent.kde.org/plasma/plasma-nm/-/commit/eb8f4ec47d7f726e52612ab440d7a39b9a785bff

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to