From 8006d62f55627db1d066f2dcf6f7f29f852d5e6c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Thu, 16 Jun 2016 00:30:31 +0200
Subject: [PATCH 01/10] Restored most keyboard shortcuts from Subsonic
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: François-Xavier Thomas <fx.thomas@gmail.com>
---
 .../src/main/webapp/WEB-INF/jsp/head.jsp      | 28 ++++++
 .../src/main/webapp/WEB-INF/jsp/playQueue.jsp | 99 +++++++++++++++++--
 .../src/main/webapp/WEB-INF/jsp/top.jsp       |  8 ++
 .../src/main/webapp/script/mousetrap-1.6.0.js | 11 +++
 4 files changed, 136 insertions(+), 10 deletions(-)
 create mode 100644 libresonic-main/src/main/webapp/script/mousetrap-1.6.0.js

diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/head.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/head.jsp
index b14d62a2..5331458b 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/head.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/head.jsp
@@ -10,3 +10,31 @@
 <link rel="stylesheet" href="https://fonts.googleapis.com/css?family=Ubuntu&subset=latin,cyrillic-ext,greek-ext,greek,latin-ext,cyrillic" type="text/css"/>
 <link rel="stylesheet" href="https://fonts.googleapis.com/css?family=Roboto:400,300,400italic,500,300italic,500italic,700,700italic,100,100italic" type="text/css"/>
 <title>Libresonic</title>
+
+<script type="text/javascript" src="<c:url value="/script/mousetrap-1.6.0.js"/>"></script>
+<script type="text/javascript">
+
+    Mousetrap.bind('space', function() { parent.frames.playQueue.onToggleStartStop(); return false; });
+    Mousetrap.bind('left',  function() { parent.frames.playQueue.onPrevious(); });
+    Mousetrap.bind('right', function() { parent.frames.playQueue.onNext(); });
+    Mousetrap.bind('*',     function() { parent.frames.playQueue.onStarCurrent(); });
+    Mousetrap.bind('plus',  function() { parent.frames.playQueue.onGainAdd(+5); });
+    Mousetrap.bind('-',     function() { parent.frames.playQueue.onGainAdd(-5); });
+    Mousetrap.bind('q',     function() { parent.frames.playQueue.onTogglePlayQueue(); });
+
+    Mousetrap.bind('/',     function() { parent.frames.upper.$("#query").focus(); });
+    Mousetrap.bind('m',     function() { parent.frames.upper.toggleLeftFrameVisible(); });
+
+    Mousetrap.bind('g h', function() { parent.frames.main.location.href = "home.view?"; });
+    Mousetrap.bind('g p', function() { parent.frames.main.location.href = "playlists.view?"; });
+    Mousetrap.bind('g o', function() { parent.frames.main.location.href = "podcastChannels.view?"; });
+    Mousetrap.bind('g s', function() { parent.frames.main.location.href = "settings.view?"; });
+    Mousetrap.bind('g t', function() { parent.frames.main.location.href = "starred.view?"; });
+    Mousetrap.bind('g r', function() { parent.frames.main.location.href = "more.view?"; });
+    Mousetrap.bind('g a', function() { parent.frames.main.location.href = "help.view?"; });
+    Mousetrap.bind('?',   function() { parent.frames.main.location.href = "more.view#shortcuts"; });
+
+    // Mousetrap.bind('shift+left', function() { getPlayerWindow().keyboardShortcut("seekBackward") });
+    // Mousetrap.bind('shift+right', function() { getPlayerWindow().keyboardShortcut("seekForward") });
+
+</script>
diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
index 7f539ddd..7e6cb2b9 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
@@ -37,6 +37,7 @@
     var songs = null;
     var currentStreamUrl = null;
     var repeatEnabled = false;
+    var isVisible = ${model.autoHidePlayQueue ? 'false' : 'true'};
     var CastPlayer = new CastPlayer();
     var ignore = false;
 
@@ -86,17 +87,30 @@
         getPlayQueue();
     }
 
+    function onHidePlayQueue() {
+      setFrameHeight(50);
+      isVisible = false;
+    }
+
+    function onShowPlayQueue() {
+      var height = $("body").height() + 25;
+      height = Math.min(height, window.top.innerHeight * 0.8);
+      setFrameHeight(height);
+      isVisible = true;
+    }
+
+    function onTogglePlayQueue() {
+      if (isVisible) onHidePlayQueue();
+      else onShowPlayQueue();
+    }
+
     function initAutoHide() {
         $(window).mouseleave(function (event) {
-            if (event.clientY < 30) {
-                setFrameHeight(50);
-            }
+            if (event.clientY < 30) onHidePlayQueue();
         });
 
         $(window).mouseenter(function () {
-            var height = $("body").height() + 25;
-            height = Math.min(height, window.top.innerHeight * 0.8);
-            setFrameHeight(height);
+            onShowPlayQueue();
         });
     }
 
@@ -155,12 +169,56 @@
             playQueueService.clear(playQueueCallback);
         }
     }
+
+    /**
+     * Start playing from the current playlist
+     */
     function onStart() {
-        playQueueService.start(playQueueCallback);
+        if (CastPlayer.castSession) {
+            CastPlayer.playCast();
+        } else if (jwplayer()) {
+            if (jwplayer().getState() == "IDLE") {
+                skip(0);
+            } else if (jwplayer().getState() == "PAUSED") {
+                jwplayer().play(true);
+            }
+        } else {
+            playQueueService.start(playQueueCallback);
+        }
     }
+
+    /**
+     * Pause playing
+     */
     function onStop() {
-        playQueueService.stop(playQueueCallback);
+        if (CastPlayer.castSession) {
+            CastPlayer.pauseCast();
+        } else if (jwplayer()) {
+            jwplayer().pause(true);
+        } else {
+            playQueueService.stop(playQueueCallback);
+        }
     }
+
+    /**
+     * Toggle play/pause
+     *
+     * FIXME: Only works for the Web player for now
+     */
+    function onToggleStartStop() {
+        var playing = false;
+        if (CastPlayer.castSession) {
+            playing = CastPlayer.mediaSession && CastPlayer.mediaSession.playerState == chrome.cast.media.PlayerState.PLAYING;
+        } else if (jwplayer()) {
+            playing = jwplayer().getState() == "PLAYING";
+        } else {
+            // FIXME for playQueueService
+        }
+
+        if (playing) onStop();
+        else onStart();
+    }
+
     function onGain(gain) {
         playQueueService.setGain(gain);
     }
@@ -172,6 +230,24 @@
         var value = parseInt($("#castVolume").slider("option", "value"));
         CastPlayer.setCastVolume(value / 100, false);
     }
+
+    /**
+     * Increase or decrease volume by a certain amount
+     *
+     * @param amount to add or remove from the current volume
+     */
+    function onGainAdd(gain) {
+        if (CastPlayer.castSession) {
+            var volume = parseInt($("#castVolume").slider("option", "value")) + gain;
+            $("#castVolume").slider("option", "value", volume);
+        } else if (jwplayer()) {
+            jwplayer().setVolume(jwplayer().getVolume() + gain);
+        } else {
+            var value = parseInt($("#jukeboxVolume").slider("option", "value"));
+            $("#jukeboxVolume").slider("option", "value", volume);
+        }
+    }
+
     function onSkip(index) {
     <c:choose>
     <c:when test="${model.player.web}">
@@ -188,10 +264,10 @@
         if (wrap) {
             index = index % songs.length;
         }
-        skip(index);
+        onSkip(index);
     }
     function onPrevious() {
-        skip(parseInt(getCurrentSongIndex()) - 1);
+        onSkip(parseInt(getCurrentSongIndex()) - 1);
     }
     function onPlay(id) {
         playQueueService.play(id, playQueueCallback);
@@ -235,6 +311,9 @@
     function onStar(index) {
         playQueueService.toggleStar(index, playQueueCallback);
     }
+    function onStarCurrent() {
+        onStar(getCurrentSongIndex());
+    }
     function onRemove(index) {
         playQueueService.remove(index, playQueueCallback);
     }
diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/top.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/top.jsp
index 7eeb095d..28791bfa 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/top.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/top.jsp
@@ -10,6 +10,7 @@
     <script type="text/javascript">
         var previousQuery = "";
         var instantSearchTimeout;
+        var showSideBar = ${model.showSideBar ? 'true' : 'false'};
 
         function triggerInstantSearch() {
             if (instantSearchTimeout) {
@@ -31,6 +32,7 @@
             $("#hide-left-frame").show();
             toggleLeftFrame(230);
             multiService.setShowSideBar(true);
+            showSideBar = true;
         }
 
         function hideLeftFrame() {
@@ -38,6 +40,12 @@
             $("#show-left-frame").show();
             toggleLeftFrame(0);
             multiService.setShowSideBar(false);
+            showSideBar = false;
+        }
+
+        function toggleLeftFrameVisible() {
+            if (showSideBar) hideLeftFrame();
+            else showLeftFrame();
         }
 
         function toggleLeftFrame(width) {
diff --git a/libresonic-main/src/main/webapp/script/mousetrap-1.6.0.js b/libresonic-main/src/main/webapp/script/mousetrap-1.6.0.js
new file mode 100644
index 00000000..01289500
--- /dev/null
+++ b/libresonic-main/src/main/webapp/script/mousetrap-1.6.0.js
@@ -0,0 +1,11 @@
+/* mousetrap v1.6.0 craig.is/killing/mice */
+(function(r,t,g){function u(a,b,h){a.addEventListener?a.addEventListener(b,h,!1):a.attachEvent("on"+b,h)}function y(a){if("keypress"==a.type){var b=String.fromCharCode(a.which);a.shiftKey||(b=b.toLowerCase());return b}return k[a.which]?k[a.which]:p[a.which]?p[a.which]:String.fromCharCode(a.which).toLowerCase()}function D(a){var b=[];a.shiftKey&&b.push("shift");a.altKey&&b.push("alt");a.ctrlKey&&b.push("ctrl");a.metaKey&&b.push("meta");return b}function v(a){return"shift"==a||"ctrl"==a||"alt"==a||
+"meta"==a}function z(a,b){var h,c,e,g=[];h=a;"+"===h?h=["+"]:(h=h.replace(/\+{2}/g,"+plus"),h=h.split("+"));for(e=0;e<h.length;++e)c=h[e],A[c]&&(c=A[c]),b&&"keypress"!=b&&B[c]&&(c=B[c],g.push("shift")),v(c)&&g.push(c);h=c;e=b;if(!e){if(!n){n={};for(var l in k)95<l&&112>l||k.hasOwnProperty(l)&&(n[k[l]]=l)}e=n[h]?"keydown":"keypress"}"keypress"==e&&g.length&&(e="keydown");return{key:c,modifiers:g,action:e}}function C(a,b){return null===a||a===t?!1:a===b?!0:C(a.parentNode,b)}function c(a){function b(a){a=
+a||{};var b=!1,m;for(m in n)a[m]?b=!0:n[m]=0;b||(w=!1)}function h(a,b,m,f,c,h){var g,e,k=[],l=m.type;if(!d._callbacks[a])return[];"keyup"==l&&v(a)&&(b=[a]);for(g=0;g<d._callbacks[a].length;++g)if(e=d._callbacks[a][g],(f||!e.seq||n[e.seq]==e.level)&&l==e.action){var q;(q="keypress"==l&&!m.metaKey&&!m.ctrlKey)||(q=e.modifiers,q=b.sort().join(",")===q.sort().join(","));q&&(q=f&&e.seq==f&&e.level==h,(!f&&e.combo==c||q)&&d._callbacks[a].splice(g,1),k.push(e))}return k}function g(a,b,m,f){d.stopCallback(b,
+b.target||b.srcElement,m,f)||!1!==a(b,m)||(b.preventDefault?b.preventDefault():b.returnValue=!1,b.stopPropagation?b.stopPropagation():b.cancelBubble=!0)}function e(a){"number"!==typeof a.which&&(a.which=a.keyCode);var b=y(a);b&&("keyup"==a.type&&x===b?x=!1:d.handleKey(b,D(a),a))}function k(a,c,m,f){function e(c){return function(){w=c;++n[a];clearTimeout(r);r=setTimeout(b,1E3)}}function h(c){g(m,c,a);"keyup"!==f&&(x=y(c));setTimeout(b,10)}for(var d=n[a]=0;d<c.length;++d){var p=d+1===c.length?h:e(f||
+z(c[d+1]).action);l(c[d],p,f,a,d)}}function l(a,b,c,f,e){d._directMap[a+":"+c]=b;a=a.replace(/\s+/g," ");var g=a.split(" ");1<g.length?k(a,g,b,c):(c=z(a,c),d._callbacks[c.key]=d._callbacks[c.key]||[],h(c.key,c.modifiers,{type:c.action},f,a,e),d._callbacks[c.key][f?"unshift":"push"]({callback:b,modifiers:c.modifiers,action:c.action,seq:f,level:e,combo:a}))}var d=this;a=a||t;if(!(d instanceof c))return new c(a);d.target=a;d._callbacks={};d._directMap={};var n={},r,x=!1,p=!1,w=!1;d._handleKey=function(a,
+c,e){var f=h(a,c,e),d;c={};var k=0,l=!1;for(d=0;d<f.length;++d)f[d].seq&&(k=Math.max(k,f[d].level));for(d=0;d<f.length;++d)f[d].seq?f[d].level==k&&(l=!0,c[f[d].seq]=1,g(f[d].callback,e,f[d].combo,f[d].seq)):l||g(f[d].callback,e,f[d].combo);f="keypress"==e.type&&p;e.type!=w||v(a)||f||b(c);p=l&&"keydown"==e.type};d._bindMultiple=function(a,b,c){for(var d=0;d<a.length;++d)l(a[d],b,c)};u(a,"keypress",e);u(a,"keydown",e);u(a,"keyup",e)}if(r){var k={8:"backspace",9:"tab",13:"enter",16:"shift",17:"ctrl",
+18:"alt",20:"capslock",27:"esc",32:"space",33:"pageup",34:"pagedown",35:"end",36:"home",37:"left",38:"up",39:"right",40:"down",45:"ins",46:"del",91:"meta",93:"meta",224:"meta"},p={106:"*",107:"+",109:"-",110:".",111:"/",186:";",187:"=",188:",",189:"-",190:".",191:"/",192:"`",219:"[",220:"\\",221:"]",222:"'"},B={"~":"`","!":"1","@":"2","#":"3",$:"4","%":"5","^":"6","&":"7","*":"8","(":"9",")":"0",_:"-","+":"=",":":";",'"':"'","<":",",">":".","?":"/","|":"\\"},A={option:"alt",command:"meta","return":"enter",
+escape:"esc",plus:"+",mod:/Mac|iPod|iPhone|iPad/.test(navigator.platform)?"meta":"ctrl"},n;for(g=1;20>g;++g)k[111+g]="f"+g;for(g=0;9>=g;++g)k[g+96]=g;c.prototype.bind=function(a,b,c){a=a instanceof Array?a:[a];this._bindMultiple.call(this,a,b,c);return this};c.prototype.unbind=function(a,b){return this.bind.call(this,a,function(){},b)};c.prototype.trigger=function(a,b){if(this._directMap[a+":"+b])this._directMap[a+":"+b]({},a);return this};c.prototype.reset=function(){this._callbacks={};this._directMap=
+{};return this};c.prototype.stopCallback=function(a,b){return-1<(" "+b.className+" ").indexOf(" mousetrap ")||C(b,this.target)?!1:"INPUT"==b.tagName||"SELECT"==b.tagName||"TEXTAREA"==b.tagName||b.isContentEditable};c.prototype.handleKey=function(){return this._handleKey.apply(this,arguments)};c.addKeycodes=function(a){for(var b in a)a.hasOwnProperty(b)&&(k[b]=a[b]);n=null};c.init=function(){var a=c(t),b;for(b in a)"_"!==b.charAt(0)&&(c[b]=function(b){return function(){return a[b].apply(a,arguments)}}(b))};
+c.init();r.Mousetrap=c;"undefined"!==typeof module&&module.exports&&(module.exports=c);"function"===typeof define&&define.amd&&define(function(){return c})}})("undefined"!==typeof window?window:null,"undefined"!==typeof window?document:null);

From fccf2a013d913de636f19d1d52c523a9e512f4bd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Sat, 18 Jun 2016 21:16:18 +0200
Subject: [PATCH 02/10] Add user option for keyboard shortcuts

---
 .../command/PersonalSettingsCommand.java      |  9 +++++++++
 .../player/controller/MultiController.java    |  1 +
 .../PersonalSettingsController.java           |  2 ++
 .../org/libresonic/player/dao/UserDao.java    |  5 +++--
 .../player/dao/schema/hsql/Schema61.java      |  6 ++++++
 .../player/domain/UserSettings.java           |  9 +++++++++
 .../player/service/SettingsService.java       |  1 +
 .../player/i18n/ResourceBundle_de.properties  |  1 +
 .../player/i18n/ResourceBundle_en.properties  |  1 +
 .../src/main/webapp/WEB-INF/jsp/head.jsp      | 19 ++++++++++++++-----
 .../webapp/WEB-INF/jsp/personalSettings.jsp   |  9 ++++++++-
 11 files changed, 55 insertions(+), 8 deletions(-)

diff --git a/libresonic-main/src/main/java/org/libresonic/player/command/PersonalSettingsCommand.java b/libresonic-main/src/main/java/org/libresonic/player/command/PersonalSettingsCommand.java
index bea8cd23..1f182583 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/command/PersonalSettingsCommand.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/command/PersonalSettingsCommand.java
@@ -52,6 +52,7 @@ public class PersonalSettingsCommand {
     private boolean showArtistInfoEnabled;
     private boolean nowPlayingAllowed;
     private boolean autoHidePlayQueue;
+    private boolean keyboardShortcutsEnabled;
     private boolean finalVersionNotificationEnabled;
     private boolean betaVersionNotificationEnabled;
     private boolean songNotificationEnabled;
@@ -230,6 +231,14 @@ public class PersonalSettingsCommand {
         this.autoHidePlayQueue = autoHidePlayQueue;
     }
 
+    public boolean isKeyboardShortcutsEnabled() {
+        return keyboardShortcutsEnabled;
+    }
+
+    public void setKeyboardShortcutsEnabled(boolean keyboardShortcutsEnabled) {
+        this.keyboardShortcutsEnabled = keyboardShortcutsEnabled;
+    }
+
     public boolean isLastFmEnabled() {
         return lastFmEnabled;
     }
diff --git a/libresonic-main/src/main/java/org/libresonic/player/controller/MultiController.java b/libresonic-main/src/main/java/org/libresonic/player/controller/MultiController.java
index e37b736e..c332e2e4 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/controller/MultiController.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/controller/MultiController.java
@@ -238,6 +238,7 @@ public class MultiController extends MultiActionController {
         map.put("showRight", userSettings.isShowNowPlayingEnabled() || userSettings.isShowChatEnabled());
         map.put("autoHidePlayQueue", userSettings.isAutoHidePlayQueue());
         map.put("listReloadDelay", userSettings.getListReloadDelay());
+        map.put("keyboardShortcutsEnabled", userSettings.isKeyboardShortcutsEnabled());
         map.put("showSideBar", userSettings.isShowSideBar());
         map.put("brand", settingsService.getBrand());
         return new ModelAndView("index", "model", map);
diff --git a/libresonic-main/src/main/java/org/libresonic/player/controller/PersonalSettingsController.java b/libresonic-main/src/main/java/org/libresonic/player/controller/PersonalSettingsController.java
index e772c224..3a0cb5f0 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/controller/PersonalSettingsController.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/controller/PersonalSettingsController.java
@@ -74,6 +74,7 @@ public class PersonalSettingsController extends SimpleFormController {
         command.setSongNotificationEnabled(userSettings.isSongNotificationEnabled());
         command.setAutoHidePlayQueue(userSettings.isAutoHidePlayQueue());
         command.setListReloadDelay(userSettings.getListReloadDelay());
+        command.setKeyboardShortcutsEnabled(userSettings.isKeyboardShortcutsEnabled());
         command.setLastFmEnabled(userSettings.isLastFmEnabled());
         command.setLastFmUsername(userSettings.getLastFmUsername());
         command.setLastFmPassword(userSettings.getLastFmPassword());
@@ -137,6 +138,7 @@ public class PersonalSettingsController extends SimpleFormController {
         settings.setSongNotificationEnabled(command.isSongNotificationEnabled());
         settings.setAutoHidePlayQueue(command.isAutoHidePlayQueue());
         settings.setListReloadDelay(command.getListReloadDelay());
+        settings.setKeyboardShortcutsEnabled(command.isKeyboardShortcutsEnabled());
         settings.setLastFmEnabled(command.isLastFmEnabled());
         settings.setLastFmUsername(command.getLastFmUsername());
         settings.setSystemAvatarId(getSystemAvatarId(command));
diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/UserDao.java b/libresonic-main/src/main/java/org/libresonic/player/dao/UserDao.java
index 87c3d81a..b8aa5e45 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/dao/UserDao.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/dao/UserDao.java
@@ -44,7 +44,7 @@ public class UserDao extends AbstractDao {
             "playlist_year, playlist_bit_rate, playlist_duration, playlist_format, playlist_file_size, " +
             "last_fm_enabled, last_fm_username, last_fm_password, transcode_scheme, show_now_playing, selected_music_folder_id, " +
             "party_mode_enabled, now_playing_allowed, avatar_scheme, system_avatar_id, changed, show_chat, show_artist_info, auto_hide_play_queue, " +
-            "view_as_list, default_album_list, queue_following_songs, show_side_bar, list_reload_delay";
+            "view_as_list, default_album_list, queue_following_songs, show_side_bar, list_reload_delay, keyboard_shortcuts_enabled";
 
     private static final Integer ROLE_ID_ADMIN = 1;
     private static final Integer ROLE_ID_DOWNLOAD = 2;
@@ -188,7 +188,7 @@ public class UserDao extends AbstractDao {
                                                    settings.getAvatarScheme().name(), settings.getSystemAvatarId(), settings.getChanged(),
                                                    settings.isShowChatEnabled(), settings.isShowArtistInfoEnabled(), settings.isAutoHidePlayQueue(),
                                                    settings.isViewAsList(), settings.getDefaultAlbumList().getId(), settings.isQueueFollowingSongs(),
-                                                   settings.isShowSideBar(), settings.getListReloadDelay()});
+                                                   settings.isShowSideBar(), settings.getListReloadDelay(), settings.isKeyboardShortcutsEnabled()});
     }
 
     private static String encrypt(String s) {
@@ -350,6 +350,7 @@ public class UserDao extends AbstractDao {
             settings.setQueueFollowingSongs(rs.getBoolean(col++));
             settings.setShowSideBar(rs.getBoolean(col++));
             settings.setListReloadDelay((Integer) rs.getObject(col++));
+            settings.setKeyboardShortcutsEnabled(rs.getBoolean(col++));
 
             return settings;
         }
diff --git a/libresonic-main/src/main/java/org/libresonic/player/dao/schema/hsql/Schema61.java b/libresonic-main/src/main/java/org/libresonic/player/dao/schema/hsql/Schema61.java
index d2634efa..378d11c3 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/dao/schema/hsql/Schema61.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/dao/schema/hsql/Schema61.java
@@ -44,5 +44,11 @@ public class Schema61 extends Schema {
             template.execute("alter table user_settings add list_reload_delay int default 60 not null");
             LOG.info("Database column 'user_settings.list_reload_delay' was added successfully.");
         }
+
+        if (!columnExists(template, "keyboard_shortcuts_enabled", "user_settings")) {
+            LOG.info("Database column 'user_settings.keyboard_shortcuts_enabled' not found.  Creating it.");
+            template.execute("alter table user_settings add keyboard_shortcuts_enabled boolean default false not null");
+            LOG.info("Database column 'user_settings.keyboard_shortcuts_enabled' was added successfully.");
+        }
     }
 }
diff --git a/libresonic-main/src/main/java/org/libresonic/player/domain/UserSettings.java b/libresonic-main/src/main/java/org/libresonic/player/domain/UserSettings.java
index 05a9139c..3d88454a 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/domain/UserSettings.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/domain/UserSettings.java
@@ -38,6 +38,7 @@ public class UserSettings {
     private boolean finalVersionNotificationEnabled;
     private boolean betaVersionNotificationEnabled;
     private boolean songNotificationEnabled;
+    private boolean keyboardShortcutsEnabled;
     private boolean autoHidePlayQueue;
     private boolean showSideBar;
     private boolean viewAsList;
@@ -221,6 +222,14 @@ public class UserSettings {
         this.autoHidePlayQueue = autoHidePlayQueue;
     }
 
+    public boolean isKeyboardShortcutsEnabled() {
+        return keyboardShortcutsEnabled;
+    }
+
+    public void setKeyboardShortcutsEnabled(boolean keyboardShortcutsEnabled) {
+        this.keyboardShortcutsEnabled = keyboardShortcutsEnabled;
+    }
+
     public boolean isShowSideBar() {
         return showSideBar;
     }
diff --git a/libresonic-main/src/main/java/org/libresonic/player/service/SettingsService.java b/libresonic-main/src/main/java/org/libresonic/player/service/SettingsService.java
index 2c579baf..4ca1eeef 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/service/SettingsService.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/service/SettingsService.java
@@ -1264,6 +1264,7 @@ public class SettingsService {
         settings.setPartyModeEnabled(false);
         settings.setNowPlayingAllowed(true);
         settings.setAutoHidePlayQueue(true);
+        settings.setKeyboardShortcutsEnabled(false);
         settings.setShowSideBar(true);
         settings.setShowArtistInfoEnabled(true);
         settings.setViewAsList(false);
diff --git a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_de.properties b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_de.properties
index 5cf701c7..b7da1d8a 100644
--- a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_de.properties
+++ b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_de.properties
@@ -350,6 +350,7 @@ personalsettings.nowplayingallowed = Zeige anderen was ich h\u00F6re
 personalsettings.showchat = Zeige Chat Nachrichten
 personalsettings.showartistinfo = Zeige Artisten-Info
 personalsettings.autohideplayqueue = Playlist automatisch ausblenden
+personalsettings.keyboardshortcutsenabled = Tastaturk\u00FCrzeln aktivieren
 personalsettings.finalversionnotification = Informiere mich \u00FCber neue Versionen
 personalsettings.betaversionnotification = Informiere mich \u00FCber neue Beta Versionen
 personalsettings.songnotification = Benachrichtigen Sie mich, wenn neue Songs gespielt werden (nicht von allen Browsern unterst\u00FCtzt)
diff --git a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
index a71e11bb..510018ff 100644
--- a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
+++ b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
@@ -375,6 +375,7 @@ personalsettings.nowplayingallowed = Let others see what I am playing
 personalsettings.showchat = Show chat messages
 personalsettings.showartistinfo = Show artist info
 personalsettings.autohideplayqueue = Auto-hide play queue
+personalsettings.keyboardshortcutsenabled = Enable keyboard shortcuts
 personalsettings.finalversionnotification = Notify me about new versions
 personalsettings.betaversionnotification = Notify me about new beta versions
 personalsettings.songnotification = Notify me when new songs are played (not supported by all browsers)
diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/head.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/head.jsp
index 5331458b..45dea935 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/head.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/head.jsp
@@ -11,9 +11,21 @@
 <link rel="stylesheet" href="https://fonts.googleapis.com/css?family=Roboto:400,300,400italic,500,300italic,500italic,700,700italic,100,100italic" type="text/css"/>
 <title>Libresonic</title>
 
+<!-- Import Mousetrap and enable shortcuts if necessary -->
+<script>
+function isKeyboardShortcutsEnabled() {
+    if (window === parent.frames.top) {
+        return ${model.keyboardShortcutsEnabled ? 'true' : 'false'};
+    } else {
+        return parent.frames.top.isKeyboardShortcutsEnabled();
+    }
+}
+</script>
+
+<!-- Bind shortcuts if enabled -->
 <script type="text/javascript" src="<c:url value="/script/mousetrap-1.6.0.js"/>"></script>
 <script type="text/javascript">
-
+if (isKeyboardShortcutsEnabled()) {
     Mousetrap.bind('space', function() { parent.frames.playQueue.onToggleStartStop(); return false; });
     Mousetrap.bind('left',  function() { parent.frames.playQueue.onPrevious(); });
     Mousetrap.bind('right', function() { parent.frames.playQueue.onNext(); });
@@ -33,8 +45,5 @@
     Mousetrap.bind('g r', function() { parent.frames.main.location.href = "more.view?"; });
     Mousetrap.bind('g a', function() { parent.frames.main.location.href = "help.view?"; });
     Mousetrap.bind('?',   function() { parent.frames.main.location.href = "more.view#shortcuts"; });
-
-    // Mousetrap.bind('shift+left', function() { getPlayerWindow().keyboardShortcut("seekBackward") });
-    // Mousetrap.bind('shift+right', function() { getPlayerWindow().keyboardShortcut("seekForward") });
-
+}
 </script>
diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/personalSettings.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/personalSettings.jsp
index 449e18f6..f57974d3 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/personalSettings.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/personalSettings.jsp
@@ -179,6 +179,13 @@
         </tr>
     </table>
 
+    <table class="indent">
+        <tr>
+            <td><form:checkbox path="keyboardShortcutsEnabled" id="keyboardShortcutsEnabled" cssClass="checkbox"/></td>
+            <td><label for="keyboardShortcutsEnabled"><fmt:message key="personalsettings.keyboardshortcutsenabled"/></label></td>
+        </tr>
+    </table>
+
     <table class="indent">
         <tr>
             <td><fmt:message key="personalsettings.listreloaddelay"/></td>
@@ -253,4 +260,4 @@
     </script>
 </c:if>
 
-</body></html>
\ No newline at end of file
+</body></html>

From 3fdb07aa27d0e656eb181a3e0fb91a0fb238bf1c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Sun, 19 Jun 2016 00:07:03 +0200
Subject: [PATCH 03/10] Add keyboard shortcuts help page in more.jsp

---
 .../player/i18n/ResourceBundle_en.properties  |  21 +++++++
 .../player/theme/default_dark.properties      |   1 +
 .../player/theme/default_light.properties     |   1 +
 .../src/main/webapp/WEB-INF/jsp/more.jsp      |  53 +++++++++++++++++-
 .../webapp/icons/default_dark/keyboard.png    | Bin 0 -> 347 bytes
 .../webapp/icons/default_light/keyboard.png   | Bin 0 -> 355 bytes
 6 files changed, 74 insertions(+), 2 deletions(-)
 create mode 100644 libresonic-main/src/main/webapp/icons/default_dark/keyboard.png
 create mode 100644 libresonic-main/src/main/webapp/icons/default_light/keyboard.png

diff --git a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
index 510018ff..99fbd01e 100644
--- a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
+++ b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
@@ -248,6 +248,27 @@ more.jamstash.text = <p>Jamstash is an HTML5 player for Libresonic. <a href="{0}
                     Also available as a <a href="https://chrome.google.com/webstore/detail/jamstash/jccdpflnecheidefpofmlblgebobbloc" target="_blank">Chrome App</a>.</p>
 more.status.title = Network Status
 more.status.text = <a href="status.view">Monitor</a> the real-time status of all network media streams.
+more.keyboard.title = Keyboard Shortcuts
+more.keyboard.text = Keyboard shortcuts can be enabled in the user preferences page. Supported shortcuts are below.
+more.keyboard.playback = Playback
+more.keyboard.navigation = Navigation
+more.keyboard.general = General
+more.keyboard.playpause = Play or pause music
+more.keyboard.previous = Go to the previous song
+more.keyboard.next = Go to the next song
+more.keyboard.volumedown = Decrease the player volume
+more.keyboard.volumeup = Increase the player volume
+more.keyboard.home = Go to Home
+more.keyboard.playlists = Go to Playlists
+more.keyboard.podcasts = Go to Podcasts
+more.keyboard.settings = Go to Settings
+more.keyboard.starred = Go to Starred
+more.keyboard.more = Go to More
+more.keyboard.about = Go to About
+more.keyboard.search = Search
+more.keyboard.sidebar = Toggle left sidebar
+more.keyboard.shortcuts = Show keyboard shortcuts
+more.keyboard.then = then
 more.mobile.title = Mobile phone
 more.mobile.text = <p>You can control {0} from any WAP-enabled mobile phone or PDA.<br> \
                    Simply visit the following URL from your phone: <b>http://yourhostname/wap</b></p> \
diff --git a/libresonic-main/src/main/resources/org/libresonic/player/theme/default_dark.properties b/libresonic-main/src/main/resources/org/libresonic/player/theme/default_dark.properties
index 7c9e67aa..397c8824 100644
--- a/libresonic-main/src/main/resources/org/libresonic/player/theme/default_dark.properties
+++ b/libresonic-main/src/main/resources/org/libresonic/player/theme/default_dark.properties
@@ -21,6 +21,7 @@ helpImage = icons/default_dark/help.png
 helpPopupImage = icons/default_dark/help_small.png
 homeImage = icons/default_dark/home.png
 html5Image = icons/default_dark/html5.png
+keyboardImage = icons/default_dark/keyboard.png
 logImage = icons/default_dark/log.png
 logoImage = icons/default_dark/logo_light.png
 moreImage = icons/default_dark/more.png
diff --git a/libresonic-main/src/main/resources/org/libresonic/player/theme/default_light.properties b/libresonic-main/src/main/resources/org/libresonic/player/theme/default_light.properties
index 3c346cd9..92ee2559 100644
--- a/libresonic-main/src/main/resources/org/libresonic/player/theme/default_light.properties
+++ b/libresonic-main/src/main/resources/org/libresonic/player/theme/default_light.properties
@@ -21,6 +21,7 @@ helpImage = icons/default_light/help.png
 helpPopupImage = icons/default_light/help_small.png
 homeImage = icons/default_light/home.png
 html5Image = icons/default_light/html5.png
+keyboardImage = icons/default_light/keyboard.png
 logImage = icons/default_light/log.png
 logoImage = icons/default_light/logo.png
 moreImage = icons/default_light/more.png
diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
index 1f0e016c..79360847 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
@@ -182,9 +182,58 @@
     <p class="detail" id="progressText"/>
 
     <div id="progressBar">
-        <div id="progressBarContent"/>
+        <div id="progressBarContent"></div>
     </div>
 
 </c:if>
 
-</body></html>
\ No newline at end of file
+<a name="shortcuts"></a>
+<h2>
+    <img src="<spring:theme code="keyboardImage"/>" alt=""/>
+    <span style="vertical-align: middle"><fmt:message key="more.keyboard.title"/></span>
+</h2>
+<fmt:message key="more.keyboard.text"/>
+<table class="indent music" style="width:inherit">
+    <tr>
+        <th colspan="2"><fmt:message key="more.keyboard.playback"/></th>
+        <th colspan="2"><fmt:message key="more.keyboard.navigation"/></th>
+        <th colspan="2"><fmt:message key="more.keyboard.general"/></th>
+    </tr>
+    <tr>
+        <td class="more-shortcut">Space</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.playpause"/></td>
+        <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> h</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.home"/></td>
+        <td class="more-shortcut">/</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.search"/></td>
+    </tr>
+    <tr>
+        <td class="more-shortcut">&#8592;</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.previous"/></td>
+        <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> p</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.playlists"/></td>
+        <td class="more-shortcut">m</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.sidebar"/></td>
+    </tr>
+    <tr>
+        <td class="more-shortcut">&#8593;</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.next"/></td>
+        <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> o</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.podcasts"/></td>
+        <td class="more-shortcut">?</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.shortcuts"/></td>
+    </tr>
+    <tr>
+        <td class="more-shortcut">&ndash;</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.volumedown"/></td>
+        <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> s</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.settings"/></td>
+        <td></td><td></td>
+    </tr>
+    <tr>
+        <td class="more-shortcut">+</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.volumeup"/></td>
+        <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> t</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.starred"/></td>
+        <td></td><td></td>
+    </tr>
+    <tr>
+        <td></td><td></td>
+        <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> r</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.more"/></td>
+        <td></td><td></td>
+    </tr>
+    <tr>
+        <td></td><td></td>
+        <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> a</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.about"/></td>
+        <td></td><td></td>
+    </tr>
+</table>
+
+</body></html>
diff --git a/libresonic-main/src/main/webapp/icons/default_dark/keyboard.png b/libresonic-main/src/main/webapp/icons/default_dark/keyboard.png
new file mode 100644
index 0000000000000000000000000000000000000000..2209f8e4eee99ce32f3de016165ca4f1706afe6c
GIT binary patch
literal 347
zcmeAS@N?(olHy`uVBq!ia0vp^3LwnE1|*BCs=fdzwj^(N7l!{JxM1({$v_d#0*}aI
z1_o|n5N2eUHAey{$X?><>&pIsO-R&8tNEGwDxlB{PZ!6Kh{JEED+(Pl5ODqf{z?Hy
z&Q4~rSQD{}(gCMg0}`}1ya-XYWQj>L)I40WjJH)SI&ITC|4BE~zCT^gm{h?hxboLA
z;qxb-ue0oY&h*CAyS`+qJj094*HyUR%PS~JHeB*@-o)p(v*XGAb@Hr-;>&z@cS|+|
zC-~{BtQVOm+R(cyxbsAqcKP80zN;BBuScpy@?2n_)bDnCqs)O_*|{3pe8oJ+8w@U=
znp^kix$RzamDgeK<FDm><vXzGbzcXM&veG={1$r4|9@=Uvh=LOUGeYNYK%SEW{FMV
nIywEly~<P}rsqFOx<9hyzg}U~d+NR+(9aB>u6{1-oD!M<1<Z#T

literal 0
HcmV?d00001

diff --git a/libresonic-main/src/main/webapp/icons/default_light/keyboard.png b/libresonic-main/src/main/webapp/icons/default_light/keyboard.png
new file mode 100644
index 0000000000000000000000000000000000000000..b542e5c51a197b8a6b62b71e2f40cbacef444f0b
GIT binary patch
literal 355
zcmeAS@N?(olHy`uVBq!ia0vp^3LwnE1|*BCs=fdzwj^(N7l!{JxM1({$v_d#0*}aI
z1_o|n5N2eUHAey{$X?><>&pIsO-NKmtuZ`lEl}u#r;B4q#NoGLhC)pS0x=R&@`<bY
zcbvNxm2H23f5}(ouv<YFqS|i-?XGRucTCNIN7-e9ibc^~zezWr<mDG15Sixea%tPw
z3r0WhY*F7br=j`$mH9p|*&7a|t@S*1O<zHYo1wF<E9KtP8j}N$C)*uq?%JSV8ObC#
zlhMNZ+RN4_rUDa98|JRh>ooZ}H|5!3Rc_upHE9oW^(XgOKTtlI-uU$v+ktH_*Jx{(
zU2co#cHBKv{M`#-`}~!Am1bVgijUfSteWBUGW(=U2P#YxJ~1AC{>9oZU_N_@`yH|V
xTXm*hye(5#9y9&9%4))rc`6eUyY}~szb~k1e>8E=KcL?kJYD@<);T3K0RZplk$3<A

literal 0
HcmV?d00001


From 4f2887e60af6afa89ac5f0f087d603fbc0a1e19e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Sun, 19 Jun 2016 00:16:59 +0200
Subject: [PATCH 04/10] Fix issue with volume up/down shortcut

---
 libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
index 7e6cb2b9..447ac3dc 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
@@ -241,7 +241,8 @@
             var volume = parseInt($("#castVolume").slider("option", "value")) + gain;
             $("#castVolume").slider("option", "value", volume);
         } else if (jwplayer()) {
-            jwplayer().setVolume(jwplayer().getVolume() + gain);
+            var volume = parseInt(jwplayer().getVolume());
+            jwplayer().setVolume(volume + gain);
         } else {
             var value = parseInt($("#jukeboxVolume").slider("option", "value"));
             $("#jukeboxVolume").slider("option", "value", volume);

From 349143cff1df6f8df338c4d04cdb18ee716dcc8c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Sun, 19 Jun 2016 00:48:31 +0200
Subject: [PATCH 05/10] Make shortcuts work in Jukebox mode

---
 .../player/ajax/PlayQueueService.java         | 16 ++++++++++
 .../src/main/webapp/WEB-INF/jsp/playQueue.jsp | 31 ++++++++++++-------
 2 files changed, 35 insertions(+), 12 deletions(-)

diff --git a/libresonic-main/src/main/java/org/libresonic/player/ajax/PlayQueueService.java b/libresonic-main/src/main/java/org/libresonic/player/ajax/PlayQueueService.java
index 73a91d69..a2496290 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/ajax/PlayQueueService.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/ajax/PlayQueueService.java
@@ -119,6 +119,22 @@ public class PlayQueueService {
         return convert(request, player, true);
     }
 
+    public PlayQueueInfo toggleStartStop() throws Exception {
+        HttpServletRequest request = WebContextFactory.get().getHttpServletRequest();
+        HttpServletResponse response = WebContextFactory.get().getHttpServletResponse();
+        return doToggleStartStop(request, response);
+    }
+
+    public PlayQueueInfo doToggleStartStop(HttpServletRequest request, HttpServletResponse response) throws Exception {
+        Player player = getCurrentPlayer(request, response);
+        if (player.getPlayQueue().getStatus() == PlayQueue.Status.STOPPED) {
+            player.getPlayQueue().setStatus(PlayQueue.Status.PLAYING);
+        } else if (player.getPlayQueue().getStatus() == PlayQueue.Status.PLAYING) {
+            player.getPlayQueue().setStatus(PlayQueue.Status.STOPPED);
+        }
+        return convert(request, player, true);
+    }
+
     public PlayQueueInfo skip(int index) throws Exception {
         HttpServletRequest request = WebContextFactory.get().getHttpServletRequest();
         HttpServletResponse response = WebContextFactory.get().getHttpServletResponse();
diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
index 447ac3dc..a60d97e3 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
@@ -206,17 +206,16 @@
      * FIXME: Only works for the Web player for now
      */
     function onToggleStartStop() {
-        var playing = false;
         if (CastPlayer.castSession) {
-            playing = CastPlayer.mediaSession && CastPlayer.mediaSession.playerState == chrome.cast.media.PlayerState.PLAYING;
+            var playing = CastPlayer.mediaSession && CastPlayer.mediaSession.playerState == chrome.cast.media.PlayerState.PLAYING;
+            if (playing) onStop();
+            else onStart();
         } else if (jwplayer()) {
-            playing = jwplayer().getState() == "PLAYING";
+            if (jwplayer().getState() == "PLAYING") onStop();
+            else onStart();
         } else {
-            // FIXME for playQueueService
+            playQueueService.toggleStartStop(playQueueCallback);
         }
-
-        if (playing) onStop();
-        else onStart();
     }
 
     function onGain(gain) {
@@ -239,13 +238,21 @@
     function onGainAdd(gain) {
         if (CastPlayer.castSession) {
             var volume = parseInt($("#castVolume").slider("option", "value")) + gain;
-            $("#castVolume").slider("option", "value", volume);
+            if (volume > 100) volume = 100;
+            if (volume < 0) volume = 0;
+            CastPlayer.setCastVolume(volume / 100, false);
+            $("#castVolume").slider("option", "value", volume); // Need to update UI
         } else if (jwplayer()) {
-            var volume = parseInt(jwplayer().getVolume());
-            jwplayer().setVolume(volume + gain);
+            var volume = parseInt(jwplayer().getVolume()) + gain;
+            if (volume > 100) volume = 100;
+            if (volume < 0) volume = 0;
+            jwplayer().setVolume(volume);
         } else {
-            var value = parseInt($("#jukeboxVolume").slider("option", "value"));
-            $("#jukeboxVolume").slider("option", "value", volume);
+            var volume = parseInt($("#jukeboxVolume").slider("option", "value")) + gain;
+            if (volume > 100) volume = 100;
+            if (volume < 0) volume = 0;
+            onGain(volume / 100);
+            $("#jukeboxVolume").slider("option", "value", volume); // Need to update UI
         }
     }
 

From 0fc35dd1b2ba8607b6433173d795f256f05963e5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Wed, 22 Jun 2016 23:21:40 +0200
Subject: [PATCH 06/10] Add shortcut help for 'toggle play queue'

---
 .../org/libresonic/player/i18n/ResourceBundle_en.properties   | 1 +
 libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp          | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
index 99fbd01e..e0347e13 100644
--- a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
+++ b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
@@ -267,6 +267,7 @@ more.keyboard.more = Go to More
 more.keyboard.about = Go to About
 more.keyboard.search = Search
 more.keyboard.sidebar = Toggle left sidebar
+more.keyboard.playqueue = Toggle play queue
 more.keyboard.shortcuts = Show keyboard shortcuts
 more.keyboard.then = then
 more.mobile.title = Mobile phone
diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
index 79360847..9d3cb293 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
@@ -212,12 +212,12 @@
     <tr>
         <td class="more-shortcut">&#8593;</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.next"/></td>
         <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> o</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.podcasts"/></td>
-        <td class="more-shortcut">?</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.shortcuts"/></td>
+        <td class="more-shortcut">q</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.playqueue"/></td>
     </tr>
     <tr>
         <td class="more-shortcut">&ndash;</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.volumedown"/></td>
         <td class="more-shortcut">g <fmt:message key="more.keyboard.then"/> s</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.settings"/></td>
-        <td></td><td></td>
+        <td class="more-shortcut">?</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.shortcuts"/></td>
     </tr>
     <tr>
         <td class="more-shortcut">+</td><td class="more-shortcut-descr"><fmt:message key="more.keyboard.volumeup"/></td>

From 04c9308a85e07fe3887977d7ef4c193d227e6f55 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Sat, 29 Oct 2016 15:18:46 +0200
Subject: [PATCH 07/10] Test: Add missing columns to UserDao test case

---
 .../java/org/libresonic/player/dao/UserDaoTestCase.java     | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/libresonic-main/src/test/java/org/libresonic/player/dao/UserDaoTestCase.java b/libresonic-main/src/test/java/org/libresonic/player/dao/UserDaoTestCase.java
index 934eb7cc..8a5eb206 100644
--- a/libresonic-main/src/test/java/org/libresonic/player/dao/UserDaoTestCase.java
+++ b/libresonic-main/src/test/java/org/libresonic/player/dao/UserDaoTestCase.java
@@ -161,6 +161,8 @@ public class UserDaoTestCase extends DaoTestCaseBase {
         assertFalse("Error in getUserSettings().", userSettings.isNowPlayingAllowed());
         assertSame("Error in getUserSettings().", AvatarScheme.NONE, userSettings.getAvatarScheme());
         assertNull("Error in getUserSettings().", userSettings.getSystemAvatarId());
+        assertEquals("Error in getUserSettings().", 0, userSettings.getListReloadDelay());
+        assertFalse("Error in getUserSettings().", userSettings.isKeyboardShortcutsEnabled());
 
         UserSettings settings = new UserSettings("sindre");
         settings.setLocale(Locale.SIMPLIFIED_CHINESE);
@@ -181,6 +183,8 @@ public class UserDaoTestCase extends DaoTestCaseBase {
         settings.setAvatarScheme(AvatarScheme.SYSTEM);
         settings.setSystemAvatarId(1);
         settings.setChanged(new Date(9412L));
+        settings.setListReloadDelay(60);
+        settings.setKeyboardShortcutsEnabled(true);
 
         userDao.updateUserSettings(settings);
         userSettings = userDao.getUserSettings("sindre");
@@ -204,6 +208,8 @@ public class UserDaoTestCase extends DaoTestCaseBase {
         assertSame("Error in getUserSettings().", AvatarScheme.SYSTEM, userSettings.getAvatarScheme());
         assertEquals("Error in getUserSettings().", 1, userSettings.getSystemAvatarId().intValue());
         assertEquals("Error in getUserSettings().", new Date(9412L), userSettings.getChanged());
+        assertEquals("Error in getUserSettings().", 60, userSettings.getListReloadDelay());
+        assertTrue("Error in getUserSettings().", userSettings.isKeyboardShortcutsEnabled());
 
         userDao.deleteUser("sindre");
         assertNull("Error in cascading delete.", userDao.getUserSettings("sindre"));

From 49f6420c44f0f84ce375f4e1a0bf8fc24a2432f8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Sun, 6 Nov 2016 17:30:27 +0100
Subject: [PATCH 08/10] Help: Add link to the user preferences page

---
 .../org/libresonic/player/i18n/ResourceBundle_en.properties     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
index e0347e13..5fac1bc9 100644
--- a/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
+++ b/libresonic-main/src/main/resources/org/libresonic/player/i18n/ResourceBundle_en.properties
@@ -249,7 +249,7 @@ more.jamstash.text = <p>Jamstash is an HTML5 player for Libresonic. <a href="{0}
 more.status.title = Network Status
 more.status.text = <a href="status.view">Monitor</a> the real-time status of all network media streams.
 more.keyboard.title = Keyboard Shortcuts
-more.keyboard.text = Keyboard shortcuts can be enabled in the user preferences page. Supported shortcuts are below.
+more.keyboard.text = Keyboard shortcuts can be enabled in the <a href="personalSettings.view">user preferences</a> page. Supported shortcuts are below.
 more.keyboard.playback = Playback
 more.keyboard.navigation = Navigation
 more.keyboard.general = General

From d2c23dfbcfb8fb5932be327f944e5fa1130919b9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Sun, 6 Nov 2016 17:31:23 +0100
Subject: [PATCH 09/10] Help: Add padding to keyboard shortcuts for readability

---
 libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
index 9d3cb293..4993717f 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/more.jsp
@@ -38,6 +38,12 @@
         }
     </script>
 
+    <style type="text/css">
+        .more-shortcut {
+            padding: 0 15px;
+        }
+    </style>
+
 </head>
 <body class="mainframe bgcolor1" onload="${model.user.uploadRole ? "refreshProgress()" : ""}">
 

From ef1a41b2a6efbdc99a7159e7a6723cf0dc331ea2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois-Xavier=20Thomas?= <fx.thomas@gmail.com>
Date: Thu, 10 Nov 2016 21:05:17 +0100
Subject: [PATCH 10/10] Fix jerky play queue animation right after loading

---
 libresonic-main/src/main/webapp/WEB-INF/jsp/index.jsp     | 2 +-
 libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/index.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/index.jsp
index 21010cf5..8cba809f 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/index.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/index.jsp
@@ -11,7 +11,7 @@
     <frameset id="mainFrameset" cols=${model.showSideBar ? "230,*" : "0,*"} border="0" framespacing="0" frameborder="0">
         <frame name="left" src="left.view?" marginwidth="0" marginheight="0" class="bgcolor2">
 
-        <frameset id="playQueueFrameset" rows=${model.autoHidePlayQueue ? "*,50" : "75%,25%"} border="0" framespacing="0" frameborder="0">
+        <frameset id="playQueueFrameset" rows=${model.autoHidePlayQueue ? "*,50" : "*,150"} border="0" framespacing="0" frameborder="0">
             <frameset cols="*,${model.showRight ? 235 : 0}" border="0" framespacing="0" frameborder="0">
                 <frame name="main" src="nowPlaying.view?" marginwidth="0" marginheight="0" class="bgcolor1">
                 <frame name="right" src="right.view?" class="bgcolor1">
diff --git a/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp b/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
index a60d97e3..c8917a44 100644
--- a/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
+++ b/libresonic-main/src/main/webapp/WEB-INF/jsp/playQueue.jsp
@@ -31,13 +31,13 @@
 
 <body class="bgcolor2 playlistframe" onload="init()">
 
-<span id="dummy-animation-target" style="max-width:50px;display: none"></span>
+<span id="dummy-animation-target" style="max-width: ${model.autoHide ? 50 : 150}px; display: none"></span>
 
 <script type="text/javascript" language="javascript">
     var songs = null;
     var currentStreamUrl = null;
     var repeatEnabled = false;
-    var isVisible = ${model.autoHidePlayQueue ? 'false' : 'true'};
+    var isVisible = ${model.autoHide ? 'false' : 'true'};
     var CastPlayer = new CastPlayer();
     var ignore = false;