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

cwo <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED
            Version|                            |6.7.6
  Fixed/Implemented|                            |
                 In|                            |
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/plasma-nm/-/commit/2ba4a
                   |                            |3ce55c32e432e9bf7c1fe248c56
                   |                            |eb1b13de

--- Comment #3 from cwo <[email protected]> ---
Git commit 2ba4a3ce55c32e432e9bf7c1fe248c56eb1b13de by Christoph Wolk.
Committed on 09/09/2026 at 07:25.
Pushed by cwo into branch 'master'.

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

M  +8    -1    applet/ConnectionItem.qml

https://invent.kde.org/plasma/plasma-nm/-/commit/2ba4a3ce55c32e432e9bf7c1fe248c56eb1b13de

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

Reply via email to