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/8139550d44 |ma/kwin/-/commit/a60a0a686d
                   |7c5b4d60ffed5bdbae5542b5313 |d7c02be4a9a10cb5f6d75a15a28
                   |293                         |d5b

--- Comment #32 from Weng Xuetian <[email protected]> ---
Git commit a60a0a686dd7c02be4a9a10cb5f6d75a15a28d5b by Xuetian Weng.
Committed on 19/09/2025 at 20:11.
Pushed by xuetianweng into branch 'Plasma/6.5'.

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/a60a0a686dd7c02be4a9a10cb5f6d75a15a28d5b

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

Reply via email to