perf: adjust candidate swipe action tigger

Author: tumuyan

app/src/main/java/com/osfans/trime/ime/core/Trime.java

@@ -442,7 +442,6 @@ public void selectLiquidKeyboard(final int tabIndex) {
         liquidKeyboard.setLand(orientation == Configuration.ORIENTATION_LANDSCAPE);
         liquidKeyboard.calcPadding(mainInputView.getWidth());
         symbolKeyboardType = liquidKeyboard.select(tabIndex);
-        updateComposing();
         tabView.updateTabWidth();
         if (inputRootBinding != null) {
           mTabRoot.setBackground(mCandidateRoot.getBackground());
@@ -451,8 +450,8 @@ public void selectLiquidKeyboard(final int tabIndex) {
       } else {
         symbolKeyboardType = SymbolKeyboardType.NO_KEY;
         symbolInputView.setVisibility(View.GONE);
-        updateComposing();
       }
+      updateComposing();
     }
     if (mainInputView != null)
       mainInputView.setVisibility(tabIndex >= 0 ? View.GONE : View.VISIBLE);

app/src/main/java/com/osfans/trime/ime/text/Composition.java

@@ -19,7 +19,6 @@
 package com.osfans.trime.ime.text;
 
 import android.annotation.SuppressLint;
-import android.annotation.TargetApi;
 import android.content.Context;
 import android.content.res.Configuration;
 import android.content.res.Resources;
@@ -146,7 +145,6 @@ public void updateDrawState(TextPaint ds) {
     }
   }
 
-  @TargetApi(21)
   public static class LetterSpacingSpan extends UnderlineSpan {
     private final float letterSpacing;
 

app/src/main/java/com/osfans/trime/ime/text/ScrollView.java

@@ -46,6 +46,9 @@ protected void onFinishInflate() {
     }
   }
 
+  private int swipeActionLimit = 200;
+  private float swipeStartX = -1;
+
   @SuppressLint("ClickableViewAccessibility")
   @Override
   public boolean onTouchEvent(@NonNull MotionEvent ev) {
@@ -73,9 +76,20 @@ public void fling(int velocityY) {
   public void commOnTouchEvent(@NonNull MotionEvent ev) {
     int action = ev.getAction();
     switch (action) {
-        /* case MotionEvent.ACTION_DOWN:
-        break; **/
+      case MotionEvent.ACTION_DOWN:
+        swipeActionLimit = Math.min(getWidth() / 4, getHeight() * 10);
+        Timber.i(
+            "commOnTouchEvent limit = "
+                + swipeActionLimit
+                + ", "
+                + getWidth() / 4
+                + ", "
+                + getHeight() * 4);
+        if (swipeActionLimit < 50) swipeActionLimit = 100;
+        break;
+
       case MotionEvent.ACTION_UP:
+        swipeStartX = -1;
         isMoving = false;
         // Fingers loose
         if (isNeedAnimation()) {
@@ -92,19 +106,39 @@ public void commOnTouchEvent(@NonNull MotionEvent ev) {
         }
 
         // TODO: 翻页后、手指抬起前，降低滑动速度增加阻尼感
-        if (inner.getLeft() > 100 && Rime.hasLeft()) {
+        if (inner.getLeft() > swipeActionLimit && Rime.hasLeft()) {
           if (pageUpAction != null) pageUpAction.run();
           if (inner.getWidth() > this.getWidth())
             scrollTo(this.getWidth() - inner.getWidth() + 400, 0);
-        } else if (inner.getWidth() - inner.getRight() > 100
-            && Trime.getService().hasCandidateExPage()) {
-          if (pageExAction != null) pageExAction.run();
-        } else if (inner.getWidth() - inner.getRight() > 100 && Rime.hasRight()) {
-          if (pageDownAction != null) pageDownAction.run();
-          if (inner.getWidth() > this.getWidth()) {
-            scrollTo(-400, 0);
-          }
         } else {
+          //          Timber.d("commOnTouchEvent "+getWidth() + "-" + inner.getWidth() +"+" +
+          // getScrollX()+", p=" +scrollEndPosition+", x="+ev.getX());
+          Timber.d(
+              "commOnTouchEvent dif=" + (swipeStartX - ev.getX()) + " limit=" + swipeActionLimit);
+
+          if (swipeStartX < 0) {
+            if (getWidth() - inner.getWidth() + getScrollX() >= 0) {
+              swipeStartX = ev.getX();
+            }
+          } else if (swipeStartX - ev.getX() > swipeActionLimit) {
+            if (Trime.getService().hasCandidateExPage()) {
+              if (pageExAction != null) pageExAction.run();
+              return;
+            } else if (Rime.hasRight()) {
+              if (pageDownAction != null) pageDownAction.run();
+              swipeStartX = -1;
+              if (inner.getWidth() > this.getWidth()) {
+                scrollTo(-swipeActionLimit, 0);
+                inner.layout(
+                    inner.getRight(),
+                    inner.getTop(),
+                    inner.getRight() + inner.getWidth(),
+                    inner.getBottom());
+              }
+              return;
+            }
+          }
+
           // When the scroll to the top or the most when it will not scroll, then move the layout.
           isNeedMove();
           // Log.d("MotionEvent "+isMoveing," d="+deltaX+" left="+left+" LR="+inner.getLeft()+",