From: Max Kellermann <max@duempel.org>
Date: Thu, 18 Sep 2008 10:32:09 +0000 (+0200)
Subject: screen: don't handle CMD_PLAY
X-Git-Tag: v0.12_alpha1~243
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=ccf19a7243df12b3e1b2055661b8da5925222407;p=ncmpc.git

screen: don't handle CMD_PLAY

CMD_PLAY is mapped to the Enter key by default.  Some screens
(mis-)use it for navigation, for entering a directory.  The playlist
screen actually plays the file under the cursor.  On all other
screens, pressing Enter may be confusing, since it is more of a
"confirmation" key instead of "play".  We should probaby add
CMD_CONFIRM later, but for now, disable the general CMD_PLAY handler
in screen.c.  This fixes bug 240.
---

diff --git a/src/screen.c b/src/screen.c
index e0fa3e1..62781fb 100644
--- a/src/screen.c
+++ b/src/screen.c
@@ -810,9 +810,11 @@ screen_client_cmd(mpdclient_t *c, command_t cmd)
 		return 0;
 
 	switch(cmd) {
+		/*
 	case CMD_PLAY:
 		mpdclient_cmd_play(c, MPD_PLAY_AT_BEGINNING);
 		break;
+		*/
 	case CMD_PAUSE:
 		mpdclient_cmd_pause(c, !IS_PAUSED(c->status->state));
 		break;