[uim commit] r5007 - trunk/qt4/immodule

[codesite-noreply]

Author: yamakenz
Date: Sat Sep 22 07:13:02 2007
New Revision: 5007

Modified:
   trunk/qt4/immodule/quiminputcontext.cpp

Log:
* qt4/immodule/quiminputcontext.cpp
  - (QUimInputContext::getPreeditAttrs): Fix lacking microfocus
    setting on 'selected' cursor-attr segment. Thanks No.861 of
    [uim thread 7] for the report


Modified: trunk/qt4/immodule/quiminputcontext.cpp
==============================================================================
--- trunk/qt4/immodule/quiminputcontext.cpp     (original)
+++ trunk/qt4/immodule/quiminputcontext.cpp     Sat Sep 22 07:13:02 2007
@@ -690,17 +690,15 @@
        QTextCharFormat segFmt;
 
         if ( uimAttr & UPreeditAttr_Cursor ) {
-           if ( segStrLen ) {
-               QInputMethodEvent::Attribute dummyCursor( 
QInputMethodEvent::Cursor,
-                                                         segStrLen,
-                                                         HIDE_CARET, DUMMY );
-               attrs << dummyCursor;
-           } else {
-               QInputMethodEvent::Attribute cursor( QInputMethodEvent::Cursor,
-                                                    segPos,
-                                                    SHOW_CARET, DUMMY );
-               attrs << cursor;
-           }
+           // Transparent cursor is required to set microfocus even
+           // if the caret is invisible to user.
+           //
+           // FIXME: Segment string may be outframed if the cursor is
+           // located near the right frame.
+           int visibility = ( segStrLen ) ? HIDE_CARET : SHOW_CARET;
+           QInputMethodEvent::Attribute cursor( QInputMethodEvent::Cursor,
+                                                segPos, visibility, DUMMY );
+           attrs << cursor;
        } else if ( uimAttr & UPreeditAttr_Separator ) {
            if ( !segStrLen )
                segStrLen = QString( DEFAULT_SEPARATOR_STR ).length();