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

Weng Xuetian <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/kwin/-/commit/a60a0a686d |ma/kwin/-/commit/7926c65c66
                   |d7c02be4a9a10cb5f6d75a15a28 |3af4cd415c29816a8575f72f6a8
                   |d5b                         |720

--- Comment #34 from Weng Xuetian <[email protected]> ---
Git commit 7926c65c663af4cd415c29816a8575f72f6a8720 by Xuetian Weng.
Committed on 20/09/2025 at 15:38.
Pushed by xuetianweng into branch 'Plasma/6.4'.

Fix a few potential missing opportunity that input method active state is not
synced.

1. InternalInputMethodContext::setFocusObject(nullptr) should still notify
   enabledChanged (no focus -> disable).
2. InternalInputMethodContext::update(Qt::ImEnabled) should notify
   enableChanged.
3. KWin's QPA may missing setFocusObject(nullptr) when destroy window,
   try to monitor destroyed signal as a last resort.
4. Call refreshActive() again on seat's focused text input surface
   change.
5. inputMethodAccepted should be considered as part of isEnabled
   condition. In Qt, having focus object is not equilvalent to allow input
   method.

The direct root cause to 506095 is caused by 3 & 5. Both 3 & 4 & 5 could fix
it.


(cherry picked from commit 8139550d447c5b4d60ffed5bdbae5542b5313293)

Co-authored-by: Weng Xuetian <[email protected]>

M  +6    -3    src/inputmethod.cpp
M  +20   -3    src/internalinputmethodcontext.cpp
M  +3    -0    src/internalinputmethodcontext.h

https://invent.kde.org/plasma/kwin/-/commit/7926c65c663af4cd415c29816a8575f72f6a8720

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

Reply via email to