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

Weng Xuetian <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/kwin/-/commit/2a4cdfe47b |ma/kwin/-/commit/8139550d44
                   |6bc91b299f8b619c33a932c711a |7c5b4d60ffed5bdbae5542b5313
                   |474                         |293

--- Comment #31 from Weng Xuetian <[email protected]> ---
Git commit 8139550d447c5b4d60ffed5bdbae5542b5313293 by Xuetian Weng, on behalf
of Weng Xuetian.
Committed on 19/09/2025 at 18:59.
Pushed by xuetianweng into branch 'master'.

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.

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/8139550d447c5b4d60ffed5bdbae5542b5313293

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

Reply via email to