From 82732aae116aaa7fd4b70ee23629687294e4117d Mon Sep 17 00:00:00 2001 From: Nate Goldman Date: Sat, 5 Mar 2016 09:24:29 -0800 Subject: [PATCH 1/2] fix #17 - autoresize window on play function references in hyperx need to look like: ``` onloadedmetadata=${onLoadedMetadata} ``` and not: ``` onloadedmetadata="${onLoadedMetadata}" ``` also removed some unnecessary `torrent.ready` checks in the player. --- renderer/views/player.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/renderer/views/player.js b/renderer/views/player.js index abf9d186..e16fba64 100644 --- a/renderer/views/player.js +++ b/renderer/views/player.js @@ -30,7 +30,7 @@ function Player (state, dispatch) {
@@ -79,7 +79,6 @@ function renderPlayerControls (state, dispatch) { if (state.view.devices.chromecast) { elements.push(hx` dispatch('openChromecast', torrent)}> cast @@ -88,7 +87,6 @@ function renderPlayerControls (state, dispatch) { if (state.view.devices.airplay) { elements.push(hx` dispatch('openAirplay', torrent)}> airplay @@ -96,7 +94,7 @@ function renderPlayerControls (state, dispatch) { } // On OSX, the back button is in the title bar of the window; see app.js // On other platforms, we render one over the video on mouseover - if(process.platform !== 'darwin') { + if (process.platform !== 'darwin') { elements.push(hx` dispatch('back')}> From efc116f0ad82010cde52274b847e5da2264a9d16 Mon Sep 17 00:00:00 2001 From: Nate Goldman Date: Sat, 5 Mar 2016 10:03:31 -0800 Subject: [PATCH 2/2] rm console.log --- renderer/index.js | 1 - 1 file changed, 1 deletion(-) diff --git a/renderer/index.js b/renderer/index.js index 89c63a6b..7a86a1ac 100644 --- a/renderer/index.js +++ b/renderer/index.js @@ -85,7 +85,6 @@ function init () { }) document.addEventListener('keydown', function (e) { - console.log('keydown ' + e.which) if (e.which === 27) { /* ESC means either exit fullscreen or go back */ if (state.view.isFullScreen) { dispatch('toggleFullScreen')