From f8b968c5ecd193e6884268c04dc1a3d6cf085885 Mon Sep 17 00:00:00 2001 From: Nate Goldman Date: Sat, 19 Mar 2016 20:50:52 -0700 Subject: [PATCH] fix hide on hover, always show playback position --- renderer/index.css | 20 +++++++++++--------- renderer/views/app.js | 2 +- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/renderer/index.css b/renderer/index.css index 8932b511..d70568da 100644 --- a/renderer/index.css +++ b/renderer/index.css @@ -633,13 +633,13 @@ body.drag .torrent-placeholder span { .player-controls .playback-cursor { position: absolute; - top: -8px; + top: -3px; background-color: #FFF; - width: 0; - height: 0; - border-radius: 0; - margin-top: 7px; - margin-left: 7px; + width: 3px; + height: 3px; + border-radius: 50%; + margin-top: 0; + margin-left: 0; transition-property: width, height, border-radius, margin-top, margin-left; transition-duration: 0.1s; transition-timing-function: ease-out; @@ -673,6 +673,10 @@ body.drag .torrent-placeholder span { float: right; } +.player-controls .fullscreen { + margin-right: 15px; +} + .player-controls .chromecast, .player-controls .airplay { font-size: 18px; /* make the cast icons less huge */ @@ -689,11 +693,9 @@ body.drag .torrent-placeholder span { } .player .playback-bar:hover .playback-cursor { + top: -8px; width: 14px; height: 14px; - border-radius: 7px; - margin-top: 0; - margin-left: 0; } /* diff --git a/renderer/views/app.js b/renderer/views/app.js index 3c8b0843..8f9ea424 100644 --- a/renderer/views/app.js +++ b/renderer/views/app.js @@ -21,7 +21,7 @@ function App (state, dispatch) { state.video.mouseStationarySince !== 0 && new Date().getTime() - state.video.mouseStationarySince > 2000 && !state.video.isPaused && - state.video.location === 'local' + state.playing.location === 'local' // Hide the header on Windows/Linux when in the player // On OSX, the header appears as part of the title bar