Always update ClientPreedit

The submode update hack actually doesn't clear client preedit on key
release. If we don't reset it to empty, it may be kept.

Fix fcitx#82 fcitx#83
Closes fcitx#100

src/rimeengine.cpp

@@ -723,7 +723,8 @@ RimeState *RimeEngine::state(InputContext *ic) {
     return ic->propertyFor(&factory_);
 }
 
-std::string RimeEngine::subMode(const InputMethodEntry & /*entry*/, InputContext &ic) {
+std::string RimeEngine::subMode(const InputMethodEntry & /*entry*/,
+                                InputContext &ic) {
     if (auto *rimeState = state(&ic)) {
         return rimeState->subMode();
     }

src/rimestate.cpp

@@ -286,11 +286,12 @@ void RimeState::updatePreedit(InputContext *ic, const RimeContext &context) {
     case PreeditMode::No:
         ic->inputPanel().setPreedit(preeditFromRimeContext(
             context, TextFormatFlag::NoFlag, TextFormatFlag::NoFlag));
+        ic->inputPanel().setClientPreedit(Text());
         break;
     case PreeditMode::CommitPreview: {
         ic->inputPanel().setPreedit(preeditFromRimeContext(
             context, TextFormatFlag::NoFlag, TextFormatFlag::NoFlag));
-        if (context.commit_text_preview) {
+        if (context.composition.length > 0 && context.commit_text_preview) {
             Text clientPreedit;
             clientPreedit.append(context.commit_text_preview,
                                  TextFormatFlag::Underline);
@@ -300,6 +301,8 @@ void RimeState::updatePreedit(InputContext *ic, const RimeContext &context) {
                 clientPreedit.setCursor(clientPreedit.textLength());
             }
             ic->inputPanel().setClientPreedit(clientPreedit);
+        } else {
+            ic->inputPanel().setClientPreedit(Text());
         }
     } break;
     case PreeditMode::ComposingText: {
@@ -381,7 +384,7 @@ void RimeState::commitPreedit(InputContext *ic) {
         if (!api->get_context(session, &context)) {
             return;
         }
-        if (context.commit_text_preview) {
+        if (context.composition.length > 0 && context.commit_text_preview) {
             ic->commitString(context.commit_text_preview);
         }
         api->free_context(&context);