From 5d1a512934f13bf01de2745239d12231bb3543cd Mon Sep 17 00:00:00 2001 From: Doris Ling Date: Tue, 28 Jun 2016 11:55:20 -0700 Subject: [PATCH] Update listener for animation in Gesture Settings. Use onClick listener instead of onTouch listener for handling playback for the animation video. Bug: 29779942 Change-Id: Ia30c0c49635fedd7f5afd4873ed1971529b99dcc --- .../settings/gestures/GesturePreference.java | 22 ++++++++----------- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/src/com/android/settings/gestures/GesturePreference.java b/src/com/android/settings/gestures/GesturePreference.java index ca2bb8be74c..76ceba41ef5 100644 --- a/src/com/android/settings/gestures/GesturePreference.java +++ b/src/com/android/settings/gestures/GesturePreference.java @@ -100,22 +100,18 @@ public final class GesturePreference extends SwitchPreference { imageView.setVisibility(View.VISIBLE); playButton.setVisibility(View.VISIBLE); - video.setOnTouchListener(new View.OnTouchListener() { + video.setOnClickListener(new View.OnClickListener() { @Override - public boolean onTouch(View v, MotionEvent event) { - if (event.getAction() == MotionEvent.ACTION_DOWN) { - if (mMediaPlayer != null) { - if (mMediaPlayer.isPlaying()) { - mMediaPlayer.pause(); - playButton.setVisibility(View.VISIBLE); - } else { - mMediaPlayer.start(); - playButton.setVisibility(View.GONE); - } + public void onClick(View v) { + if (mMediaPlayer != null) { + if (mMediaPlayer.isPlaying()) { + mMediaPlayer.pause(); + playButton.setVisibility(View.VISIBLE); + } else { + mMediaPlayer.start(); + playButton.setVisibility(View.GONE); } - return true; } - return false; } });