vcl/win/window/salframe.cxx |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 2b2d1c08c94fdc3982971c2b19ea241f05e578c9
Author:     Mark Hung <mark...@gmail.com>
AuthorDate: Mon Apr 4 20:12:36 2022 +0800
Commit:     Mark Hung <mark...@gmail.com>
CommitDate: Fri Apr 8 15:23:54 2022 +0200

    tdf#143447 ignore IME candidate window state.
    
    Some traditional Chinese IMEs ( Input Method Editors ) under
    Windows10 send WM_IME_NOTIFY with IMN_OPENCANDIDATE but not
    IMN_CLOSECANDIDATE. The behavior is different if users configure
    the IME and enable "legacy" option.
    
    That caused the cursor been hidden, misled by mbCandidateMode.
    
    The patch ignores the candidate window mode in case the length
    of the composition string is 0, assume in that case candidate
    window is useless, and resets the candidate window mode when
    composition ended to maintain the state as much as we can.
    
    Change-Id: I91a1c23ee1a031313243e032653f50f39b0f2a3c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/132512
    Tested-by: Jenkins
    Reviewed-by: Mark Hung <mark...@gmail.com>

diff --git a/vcl/win/window/salframe.cxx b/vcl/win/window/salframe.cxx
index cd9449c1a4ae..bbfd7843502a 100644
--- a/vcl/win/window/salframe.cxx
+++ b/vcl/win/window/salframe.cxx
@@ -5153,7 +5153,7 @@ static bool ImplHandleIMECompositionInput( WinSalFrame* 
pFrame,
         if ( (nTextLen > 0) || !(lParam & GCS_RESULTSTR) )
         {
             // End the mode, if the last character is deleted
-            if ( !nTextLen && !pFrame->mbCandidateMode )
+            if ( !nTextLen )
             {
                 pFrame->CallCallback( SalEvent::ExtTextInput, &aEvt );
                 pFrame->CallCallback( SalEvent::EndExtTextInput, nullptr );
@@ -5238,7 +5238,10 @@ static bool ImplHandleIMEEndComposition( HWND hWnd )
     if ( pFrame && pFrame->mbHandleIME )
     {
         if ( pFrame->mbAtCursorIME )
+        {
+            pFrame->mbCandidateMode = false;
             bDef = false;
+        }
     }
 
     ImplSalYieldMutexRelease();

Reply via email to