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 e0fa3e1c73b03ea96b5fd22e006fcf5880c45c90..62781fb8a2bb7bb31f41d0969a7a75eeb41200e4 100644 (file)
--- 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;