From 32a3c5392faa7a003d6326bc4fd6cb9a85086b75 Mon Sep 17 00:00:00 2001
From: Ilya Grigorik <ilya@igvita.com>
Date: Sun, 27 Aug 2017 11:31:43 -0700
Subject: [PATCH] Unify controller close (x) with display shortcut

Clicking (x) in controller is equivalent to hiding it via shortcut (v).

Closes #232
---
 inject.js | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/inject.js b/inject.js
index dee90c0..f9c9799 100644
--- a/inject.js
+++ b/inject.js
@@ -127,7 +127,7 @@ chrome.extension.sendMessage({}, function(response) {
             <button data-action="slower">-</button>
             <button data-action="faster">+</button>
             <button data-action="advance" class="rw">»</button>
-            <button data-action="close" class="hideButton">x</button>
+            <button data-action="display" class="hideButton">x</button>
           </span>
         </div>
       `;
@@ -341,9 +341,6 @@ chrome.extension.sendMessage({}, function(response) {
           v.playbackRate = Number(s.toFixed(2));
         } else if (action === 'reset') {
           resetSpeed(v, 1.0);
-        } else if (action === 'close') {
-          v.classList.add('vsc-cancelled');
-          controller.remove();
         } else if (action === 'display') {
           controller.classList.add('vsc-manual');
           controller.classList.toggle('vsc-hidden');