title: Long Click on Clickable span not firing until click is released
body: <p>I need a clickable span to have both a normal click and a long click methods in my app and i found out here (<a href="https://stackoverflow.com/questions/8702573/in-android-how-can-i-register-only-long-clicks-using-a-clickablespan">In Android - How can I register only long clicks using a ClickableSpan</a>)  that i could extend the LinkMovementMethod class and the ClickableSpan class to allow to do that but currently the long and short clicks both work but for a long click instead of firing the long click action when the item has been pressed long enough it will wait until you release the item to fire. Here is my code for the extended classes:</p>
  
  <p><strong>LinkMovementClass</strong></p>
  
  <pre><code>import android.text.Layout;
  import android.text.Selection;
  import android.text.Spannable;
  import android.text.method.LinkMovementMethod;
  import android.text.method.MovementMethod;
  import android.view.MotionEvent;
  import android.widget.TextView;
  
  public class LongClickLinkMovementMethod extends LinkMovementMethod {
  
  private Long lastClickTime = 0l;
  private int lastX = 0;
  private int lastY = 0;
  @Override
  public boolean onTouchEvent(TextView widget, Spannable buffer,
                              MotionEvent event) {
      int action = event.getAction();
  
      if (action == MotionEvent.ACTION_UP ||
              action == MotionEvent.ACTION_DOWN) {
          int x = (int) event.getX();
          int y = (int) event.getY();
          lastX = x;
          lastY = y;
          int deltaX = Math.abs(x-lastX);
          int deltaY = Math.abs(y-lastY);
  
          x -= widget.getTotalPaddingLeft();
          y -= widget.getTotalPaddingTop();
  
          x += widget.getScrollX();
          y += widget.getScrollY();
  
          Layout layout = widget.getLayout();
          int line = layout.getLineForVertical(y);
          int off = layout.getOffsetForHorizontal(line, x);
  
          LongClickableSpan[] link = buffer.getSpans(off, off, LongClickableSpan.class);
  
          if (link.length != 0) {
              if (action == MotionEvent.ACTION_UP) {
                  if (System.currentTimeMillis() - lastClickTime &lt; 1000)
                      link[0].onClick(widget);
                  else if (deltaX &lt; 10 &amp;&amp; deltaY &lt; 10)
                      link[0].onLongClick(widget);
              } else if (action == MotionEvent.ACTION_DOWN) {
                  Selection.setSelection(buffer,
                          buffer.getSpanStart(link[0]),
                          buffer.getSpanEnd(link[0]));
                  lastClickTime = System.currentTimeMillis();
              }
              return true;
          }
      }
  
      return super.onTouchEvent(widget, buffer, event);
  }
  
  
  public static MovementMethod getInstance() {
      if (sInstance == null)
          sInstance = new LongClickLinkMovementMethod();
  
      return sInstance;
  }
      private static LongClickLinkMovementMethod sInstance;
  }
  </code></pre>
  
  <p><strong>LongClickableSpanClass</strong></p>
  
  <pre><code>import android.text.style.ClickableSpan;
  import android.view.View;
  
  public abstract class LongClickableSpan extends ClickableSpan {
  
      abstract public void onLongClick(View view);
  }
  </code></pre>
  
  <p><strong>Actual implementation</strong></p>
  
  <pre><code>LongClickableSpan eruptionText = new LongClickableSpan() {
  
                                  @Override
                                  public void onClick(View tvEruptions) {
                                      LinkFunctions.link_eruption_detail(getApplicationContext(),PostErupionID);
                                  }
  
                                  @Override
                                  public void onLongClick(View tvEruptions) {
                                      if(SignedInUserID != 0) {
                                          DialogFragment newFragment = new Dialogs.QuickActionsDialogFragment();
                                          // Supply num input as an argument.
                                          Bundle args = new Bundle();
                                          args.putLong("eruptionID", PostErupionID);
                                          newFragment.setArguments(args);
                                          newFragment.show(getFragmentManager(), "QuickActions");
                                      }
                                  }
  
                              };
                              ss.setSpan(eruptionText, startpos[(int) j], endpos[(int) j], Spanned.SPAN_EXCLUSIVE_EXCLUSIVE);
  </code></pre>