Long press Android

Question

I have problems with detecting long press in my custom view.

Here's the code related to this issue

final GestureDetector gestureDetector = new GestureDetector(new GestureDetector.SimpleOnGestureListener() {
    public void onLongPress(MotionEvent e) {
        Log.e("dbg_msg", "onLongPress");
    }
});

public boolean onTouchEvent(MotionEvent event) {
    return gestureDetector.onTouchEvent(event);
};

This code detects every single (short) click as long press.

When I put this code in class inherited from Activity, it works.

So why it isn't working in custom View ?

Returning true from onTouchEvent fixed this for me – Manuel Feb 23 '14 at 10:40 — Manuel, Feb 23 '14 at 10:40

Nikola Ninkovic · Accepted Answer · 2012-07-29T10:16:22.597

25

All of this code goes in your custom view class:

public static int LONG_PRESS_TIME = 500; // Time in miliseconds 

final Handler _handler = new Handler(); 
Runnable _longPressed = new Runnable() { 
    public void run() {
        Log.i("info","LongPress");
    }   
};

@Override
public boolean onTouchEvent(MotionEvent event) {
    switch(event.getAction()){
    case MotionEvent.ACTION_DOWN:
        _handler.postDelayed(_longPressed, LONG_PRESS_TIME);
        break;
    case MotionEvent.ACTION_MOVE:
        _handler.removeCallbacks(_longPressed);
        break;
    case MotionEvent.ACTION_UP:
        _handler.removeCallbacks(_longPressed);
        break;
    }
    return true;
}

edited Jul 29 '12 at 10:16

answered Jul 28 '12 at 16:04

Nikola Ninkovic

1,252
1
12
27

1

Should also call removeCallbacks on MotionEvent.ACTION_CANCEL. – kasgoku Aug 22 '14 at 14:10
2

You should remove ACTION_MOVE because it will cancel the handler. – Marian Pavel May 05 '15 at 12:51

Anh3Saigon · Answer 2 · 2012-07-28T15:40:13.143

4

I'm not sure but your GestureDetector's constructor is deprecated (here). Could you try other ones which need a context as first parameter?

Sorry I'm new so I can't post comment.

— Edited —

It seems you used another listener, this View.OnTouchListener has other onTouch() method. Could you try again?

— Edited —

Here is an example (worked for me):

...
mAnotherView.setOnTouchListener(new View.OnTouchListener() {

    @Override
    public boolean onTouch(View v, MotionEvent event) {
        return mGestureDetector.onTouchEvent(event);
    }
});

...
mGestureDetector = new GestureDetector(this, new GestureDetector.SimpleOnGestureListener() {

    @Override
    public void onLongPress(MotionEvent e) {
        // do your tasks here
    }
});

edited Jul 28 '12 at 15:40

answered Jul 28 '12 at 14:54

Anh3Saigon

199
5

How to do that, I don't get You ? – Nikola Ninkovic Jul 28 '12 at 15:33
1

You're welcome :-) I think you can answer yourself, to help other people reading your question. – Anh3Saigon Jul 28 '12 at 15:53
Please accept your answer ([hint](http://blog.stackoverflow.com/2009/01/accept-your-own-answers/)). Sorry I talk so much… – Anh3Saigon Jul 28 '12 at 16:50
It's ok, I have to wait 48 hours to do that :) – Nikola Ninkovic Jul 28 '12 at 21:44

score 4 · Answer 3 · answered Jul 28 '12 at 15:16

4

Did you enable long presses on your GestureDetector? You can enable it either using an appropriate constructor, or by calling setIsLongpressEnabled. For instance, you can do:

gestureDetector.setIsLongpressEnabled(true);

in your constructor.

answered Jul 28 '12 at 15:16

MJD

1,183
7
13

score 0 · Answer 4 · answered Feb 10 '15 at 19:19

It's better I guess..

public class Workflow extends View implements View.OnLongClickListener {

public Workflow(Context context, DisplayFeatures d) {
    super(context);

    setLongClickable(true);
    setOnLongClickListener(this);
}

@Override
public boolean onTouchEvent(MotionEvent event) {
    /* onTouchEvent should return super.onTouchEvent(event);, otherwise long click wouldn't be performed */
    return super.onTouchEvent(event);
}

@Override
public boolean onLongClick(View v) {
    Log.d("VIEW", "LONG CLICK PERFORMED!");
    return false;
}
}

Long press Android

4 Answers4

Linked