Unify controller close (x) with display shortcut

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

Closes #232
This commit is contained in:
Ilya Grigorik
2017-08-27 11:31:43 -07:00
parent daec249e3f
commit 32a3c5392f

View File

@@ -127,7 +127,7 @@ chrome.extension.sendMessage({}, function(response) {
<button data-action="slower">-</button> <button data-action="slower">-</button>
<button data-action="faster">+</button> <button data-action="faster">+</button>
<button data-action="advance" class="rw">»</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> </span>
</div> </div>
`; `;
@@ -341,9 +341,6 @@ chrome.extension.sendMessage({}, function(response) {
v.playbackRate = Number(s.toFixed(2)); v.playbackRate = Number(s.toFixed(2));
} else if (action === 'reset') { } else if (action === 'reset') {
resetSpeed(v, 1.0); resetSpeed(v, 1.0);
} else if (action === 'close') {
v.classList.add('vsc-cancelled');
controller.remove();
} else if (action === 'display') { } else if (action === 'display') {
controller.classList.add('vsc-manual'); controller.classList.add('vsc-manual');
controller.classList.toggle('vsc-hidden'); controller.classList.toggle('vsc-hidden');