From 4a3d170f84d25ea32a59a09e04bc3a8f36f108fa Mon Sep 17 00:00:00 2001
From: Ilya Grigorik <ilya@igvita.com>
Date: Thu, 22 Dec 2016 17:12:51 -0800
Subject: [PATCH] new option to hide controller by default

When enabled controller visibility in 'manual' mode: to hide/unhide the
user needs to use the configured shortcut keys.

Closes https://github.com/igrigorik/videospeed/issues/156.
---
 inject.js    | 6 ++++++
 options.html | 4 ++++
 options.js   | 4 ++++
 3 files changed, 14 insertions(+)

diff --git a/inject.js b/inject.js
index 92c740b..5adf45c 100644
--- a/inject.js
+++ b/inject.js
@@ -12,6 +12,7 @@ chrome.extension.sendMessage({}, function(response) {
       advanceKeyCode: 88,   // default: X
       displayKeyCode: 86,   // default: V
       rememberSpeed: false, // default: false
+      startHidden: false,   // default: false
       blacklist: `
         www.instagram.com
         twitter.com
@@ -33,6 +34,7 @@ chrome.extension.sendMessage({}, function(response) {
     tc.settings.displayKeyCode = Number(storage.displayKeyCode);
     tc.settings.advanceKeyCode = Number(storage.advanceKeyCode);
     tc.settings.rememberSpeed = Boolean(storage.rememberSpeed);
+    tc.settings.startHidden = Boolean(storage.startHidden);
     tc.settings.blacklist = String(storage.blacklist);
 
     initializeWhenReady(document);
@@ -94,6 +96,10 @@ chrome.extension.sendMessage({}, function(response) {
       wrapper.addEventListener('mousedown', prevent, true);
       wrapper.addEventListener('click', prevent, true);
 
+      if (tc.settings.startHidden) {
+        wrapper.classList.add('vsc-hidden');
+      }
+
       var shadow = wrapper.createShadowRoot();
       var shadowTemplate = `
         <style>
diff --git a/options.html b/options.html
index 3a1d84b..0e2ab43 100644
--- a/options.html
+++ b/options.html
@@ -36,6 +36,10 @@
         <label for="displayKeyInput">Show/hide controller</label>
         <input id="displayKeyInput" placeholder="press a key" type="text" value=""/>
       </div>
+      <div class="row">
+          <label for="rememberSpeed">Hide controller by default</label>
+          <input id="startHidden" type="checkbox"/>
+      </div>
     </section>
 
     <section>
diff --git a/options.js b/options.js
index 42d193f..ab7af96 100644
--- a/options.js
+++ b/options.js
@@ -10,6 +10,7 @@ var tcDefaults = {
   advanceKeyCode: 88,   // default: X
   displayKeyCode: 86,   // default: V
   rememberSpeed: false, // default: false
+  startHidden: false,   // default: false
   blacklist: `
     www.instagram.com
     twitter.com
@@ -98,6 +99,7 @@ function save_options() {
   var fasterKeyCode = document.getElementById('fasterKeyInput').keyCode;
   var displayKeyCode = document.getElementById('displayKeyInput').keyCode;
   var rememberSpeed = document.getElementById('rememberSpeed').checked;
+  var startHidden = document.getElementById('startHidden').checked;
   var blacklist     = document.getElementById('blacklist').value;
 
   speedStep     = isNaN(speedStep) ? tcDefaults.speedStep : Number(speedStep);
@@ -121,6 +123,7 @@ function save_options() {
     fasterKeyCode:  fasterKeyCode,
     displayKeyCode: displayKeyCode,
     rememberSpeed:  rememberSpeed,
+    startHidden:    startHidden,
     blacklist:      blacklist.replace(/^\s+|\s+$/gm,'')
   }, function() {
     // Update status to let user know options were saved.
@@ -145,6 +148,7 @@ function restore_options() {
     updateShortcutInputText('fasterKeyInput', storage.fasterKeyCode);
     updateShortcutInputText('displayKeyInput', storage.displayKeyCode);
     document.getElementById('rememberSpeed').checked = storage.rememberSpeed;
+    document.getElementById('startHidden').checked = storage.startHidden;
     document.getElementById('blacklist').value = storage.blacklist;
   });
 }