From d2b95163fb9c4223e972b8e434099aa98e9bc667 Mon Sep 17 00:00:00 2001
From: DC <dcposch@dcpos.ch>
Date: Tue, 5 Apr 2016 17:58:38 -0700
Subject: [PATCH 1/2] Show video controls immediately on mouse move

---
 renderer/index.js | 22 +++++++++++++++++++++-
 1 file changed, 21 insertions(+), 1 deletion(-)

diff --git a/renderer/index.js b/renderer/index.js
index 24541a5b..fd554ff0 100644
--- a/renderer/index.js
+++ b/renderer/index.js
@@ -144,6 +144,7 @@ function render (state) {
 
 // Calls render() to go from state -> UI, then applies to vdom to the real DOM.
 function update () {
+  showOrHidePlayerControls()
   vdomLoop.update(state)
   updateElectron()
 }
@@ -285,7 +286,7 @@ function dispatch (action, ...args) {
   }
 
   // Update the virtual-dom, unless it's just a mouse move event
-  if (action !== 'mediaMouseMoved') {
+  if (action !== 'mediaMouseMoved' || showOrHidePlayerControls()) {
     update()
   }
 }
@@ -1015,3 +1016,22 @@ function findCommonPrefix (a, b) {
   if (i === b.length) return b
   return a.substring(0, i)
 }
+
+// Hide player controls while playing video, if the mouse stays still for a while
+// Never hide the controls when:
+// * The mouse is over the controls or we're scrubbing (see CSS)
+// * The video is paused
+// * The video is playing remotely on Chromecast or Airplay
+function showOrHidePlayerControls () {
+  var hideControls = state.location.current().url === 'player' &&
+    state.playing.mouseStationarySince !== 0 &&
+    new Date().getTime() - state.playing.mouseStationarySince > 2000 &&
+    !state.playing.isPaused &&
+    state.playing.location === 'local'
+
+  if (hideControls !== state.playing.hideControls) {
+    state.playing.hideControls = hideControls
+    return true
+  }
+  return false
+}

From b31281c35e25ccbdd94866775f57da022d734f32 Mon Sep 17 00:00:00 2001
From: DC <dcposch@dcpos.ch>
Date: Tue, 5 Apr 2016 18:21:46 -0700
Subject: [PATCH 2/2] Don't load cast module when seeking

---
 renderer/index.js    | 16 +++++++++++++---
 renderer/lib/cast.js | 10 +---------
 2 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/renderer/index.js b/renderer/index.js
index fd554ff0..8faa767e 100644
--- a/renderer/index.js
+++ b/renderer/index.js
@@ -306,14 +306,14 @@ function playPause (isPaused) {
     return // Nothing to do
   }
   // Either isPaused is undefined, or it's the opposite of the current state. Toggle.
-  if (lazyLoadCast().isCasting()) {
+  if (isCasting()) {
     Cast.playPause()
   }
   state.playing.isPaused = !state.playing.isPaused
 }
 
 function jumpToTime (time) {
-  if (lazyLoadCast().isCasting()) {
+  if (isCasting()) {
     Cast.seek(time)
   } else {
     state.playing.jumpToTime = time
@@ -325,16 +325,26 @@ function changeVolume (delta) {
   setVolume(state.playing.volume + delta)
 }
 
+// TODO: never called. Either remove or make a volume control that calls it
 function setVolume (volume) {
   // check if its in [0.0 - 1.0] range
   volume = Math.max(0, Math.min(1, volume))
-  if (lazyLoadCast().isCasting()) {
+  if (isCasting()) {
     Cast.setVolume(volume)
   } else {
     state.playing.setVolume = volume
   }
 }
 
+// Checks whether we are connected and already casting
+// Returns false if we not casting (state.playing.location === 'local')
+// or if we're trying to connect but haven't yet ('chromecast-pending', etc)
+function isCasting () {
+  return state.playing.location === 'chromecast' ||
+    state.playing.location === 'airplay' ||
+    state.playing.location === 'dlna'
+}
+
 function setupIpc () {
   ipcRenderer.send('ipcReady')
 
diff --git a/renderer/lib/cast.js b/renderer/lib/cast.js
index e755855c..f53e4617 100644
--- a/renderer/lib/cast.js
+++ b/renderer/lib/cast.js
@@ -14,8 +14,7 @@ module.exports = {
   stopCasting,
   playPause,
   seek,
-  setVolume,
-  isCasting
+  setVolume
 }
 
 // Callback to notify module users when state has changed
@@ -285,13 +284,6 @@ function stoppedCasting () {
   update()
 }
 
-// Checks whether we are connected and already casting
-// Returns false if we not casting (state.playing.location === 'local')
-// or if we're trying to connect but haven't yet ('chromecast-pending', etc)
-function isCasting () {
-  return state.playing.location === 'chromecast' || state.playing.location === 'airplay' || state.playing.location === 'dlna'
-}
-
 function getDevice (location) {
   if (location && state.devices[location]) {
     return state.devices[location]