fix: TextView's long click not working

Author: xuyangxy

README.md

@@ -32,7 +32,7 @@ In your app level `build.gradle` :
 
 ```java
 dependencies {
-    compile 'com.github.xymelon:richtext:1.0.10'
+    compile 'com.github.xymelon:richtext:1.1.0'
 }
 ```
 

richtext/src/main/java/com/xycoding/richtext/LinkTouchMovementMethod.java

@@ -7,6 +7,7 @@
 import android.text.Spannable;
 import android.text.method.LinkMovementMethod;
 import android.text.method.MovementMethod;
+import android.view.GestureDetector;
 import android.view.Gravity;
 import android.view.MotionEvent;
 import android.widget.TextView;
@@ -21,11 +22,12 @@ public class LinkTouchMovementMethod extends LinkMovementMethod {
     private static volatile LinkMovementMethod sInstance;
     private ClickSpan mClickSpan;
     private float mClickDownX, mClickDownY;
-    private Path mPath = new Path();
-    private RectF mRectF = new RectF();
-    private Rect mRect = new Rect();
+    private final Path mPath = new Path();
+    private final RectF mRectF = new RectF();
+    private final Rect mRect = new Rect();
     private int mClickSpanStart;
     private int mClickSpanEnd;
+    private GestureDetector mGestureDetector;
 
     private LinkTouchMovementMethod() {
     }
@@ -41,36 +43,40 @@ public static MovementMethod getInstance() {
     }
 
     @Override
-    public boolean onTouchEvent(final TextView textView, Spannable spannable, MotionEvent event) {
-        if (event.getAction() == MotionEvent.ACTION_DOWN) {
-            mClickSpan = getPressedSpan(textView, spannable, event);
-            if (mClickSpan != null) {
-                mClickSpanStart = spannable.getSpanStart(mClickSpan);
-                mClickSpanEnd = spannable.getSpanEnd(mClickSpan);
-                calcWordCenter(textView, mClickSpanStart, mClickSpanEnd);
-            }
-        } else if (event.getAction() == MotionEvent.ACTION_MOVE) {
-            ClickSpan touchedSpan = getPressedSpan(textView, spannable, event);
-            if (mClickSpan != null && touchedSpan != mClickSpan) {
-                mClickSpan = null;
-            }
-        } else {
-            if (mClickSpan != null) {
-                if (event.getAction() == MotionEvent.ACTION_UP) {
-                    //click callback
-                    mClickSpan.onClick(textView,
-                            textView.getText().subSequence(mClickSpanStart, mClickSpanEnd).toString(),
-                            mClickDownX,
-                            mClickDownY,
-                            spannable,
-                            mClickSpanStart,
-                            mClickSpanEnd);
+    public boolean onTouchEvent(final TextView textView, final Spannable spannable, MotionEvent event) {
+        if (mGestureDetector == null) {
+            mGestureDetector = new GestureDetector(textView.getContext().getApplicationContext(), new GestureDetector.SimpleOnGestureListener() {
+                @Override
+                public boolean onSingleTapUp(MotionEvent e) {
+                    mClickSpan = getPressedSpan(textView, spannable, e);
+                    if (mClickSpan != null) {
+                        mClickSpanStart = spannable.getSpanStart(mClickSpan);
+                        mClickSpanEnd = spannable.getSpanEnd(mClickSpan);
+                        //word center
+                        calcWordCenter(textView, mClickSpanStart, mClickSpanEnd);
+                        //click callback
+                        mClickSpan.onClick(
+                                textView,
+                                textView.getText().subSequence(mClickSpanStart, mClickSpanEnd).toString(),
+                                mClickDownX,
+                                mClickDownY,
+                                spannable,
+                                mClickSpanStart,
+                                mClickSpanEnd
+                        );
+                    }
+                    textView.invalidate();
+                    return true;
                 }
-                mClickSpan = null;
-            }
+            });
+        }
+        final boolean handled = mGestureDetector.onTouchEvent(event);
+        if (event.getAction() == MotionEvent.ACTION_UP || event.getAction() == MotionEvent.ACTION_CANCEL) {
+            //Be sure to clean up, otherwise memory leaks.
+            mClickSpan = null;
+            mGestureDetector = null;
         }
-        textView.invalidate();
-        return true;
+        return handled;
     }
 
     private ClickSpan getPressedSpan(TextView textView, Spannable spannable, MotionEvent event) {