From 6ea6688a2753c68d6b91e609aec2ae90ca44cd0e Mon Sep 17 00:00:00 2001 From: Christopher Eby Date: Sun, 28 Aug 2011 04:34:15 -0500 Subject: [PATCH] Fix dpad navigation. Call super and update CoverView immediately --- .../kreed/vanilla/FullPlaybackActivity.java | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/src/org/kreed/vanilla/FullPlaybackActivity.java b/src/org/kreed/vanilla/FullPlaybackActivity.java index b104ea78..30eade89 100644 --- a/src/org/kreed/vanilla/FullPlaybackActivity.java +++ b/src/org/kreed/vanilla/FullPlaybackActivity.java @@ -248,23 +248,19 @@ public class FullPlaybackActivity extends PlaybackActivity implements SeekBar.On public boolean onKeyDown(int keyCode, KeyEvent event) { switch (keyCode) { - case KeyEvent.KEYCODE_DPAD_RIGHT: { - View button = findViewById(R.id.next); - if (button != null) - button.requestFocus(); + case KeyEvent.KEYCODE_DPAD_RIGHT: + findViewById(R.id.next).requestFocus(); mHandler.sendMessage(mHandler.obtainMessage(PlaybackActivity.MSG_SET_SONG, 1, 0)); + mCoverView.go(1); return true; - } - case KeyEvent.KEYCODE_DPAD_LEFT: { - View button = findViewById(R.id.previous); - if (button != null) - button.requestFocus(); + case KeyEvent.KEYCODE_DPAD_LEFT: + findViewById(R.id.previous).requestFocus(); mHandler.sendMessage(mHandler.obtainMessage(PlaybackActivity.MSG_SET_SONG, -1, 0)); + mCoverView.go(-1); return true; - } } - return false; + return super.onKeyDown(keyCode, event); } @Override