diff options
author | Sam Potts <sam@potts.es> | 2018-09-25 20:43:09 +1000 |
---|---|---|
committer | Sam Potts <sam@potts.es> | 2018-09-25 20:43:09 +1000 |
commit | 62d80e6b7698d200d3b97c6084f059e2ba780efe (patch) | |
tree | fd8ef2fa03faad84fde0a89c38fd669480842f68 /src/js/listeners.js | |
parent | 7dc4d9cd22a49ba4019671cf703b25b162bcfa19 (diff) | |
download | plyr-62d80e6b7698d200d3b97c6084f059e2ba780efe.tar.lz plyr-62d80e6b7698d200d3b97c6084f059e2ba780efe.tar.xz plyr-62d80e6b7698d200d3b97c6084f059e2ba780efe.zip |
Fix touch vs click issue
Diffstat (limited to 'src/js/listeners.js')
-rw-r--r-- | src/js/listeners.js | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/src/js/listeners.js b/src/js/listeners.js index 5fe20695..138fa1d8 100644 --- a/src/js/listeners.js +++ b/src/js/listeners.js @@ -45,7 +45,7 @@ class Listeners { // Seek by the number keys const seekByKey = () => { // Divide the max duration into 10th's and times by the number value - player.currentTime = player.duration / 10 * (code - 48); + player.currentTime = (player.duration / 10) * (code - 48); }; // Handle the key on keydown @@ -146,7 +146,7 @@ class Listeners { player.loop = !player.loop; break; - /* case 73: + /* case 73: this.setLoop('start'); break; @@ -372,7 +372,7 @@ class Listeners { } // On click play, pause ore restart - on.call(player, elements.container, 'click touchstart', event => { + on.call(player, elements.container, 'click', event => { const targets = [elements.container, wrapper]; // Ignore if click if not container or in video wrapper @@ -380,13 +380,8 @@ class Listeners { return; } - // First touch on touch devices will just show controls (if we're hiding controls) - // If controls are shown then it'll toggle like a pointer device - if ( - player.config.hideControls && - player.touch && - hasClass(elements.container, player.config.classNames.hideControls) - ) { + // Touch devices will just show controls (if hidden) + if (player.touch && player.config.hideControls) { return; } @@ -596,7 +591,7 @@ class Listeners { // Set range input alternative "value", which matches the tooltip time (#954) this.bind(elements.inputs.seek, 'mousedown mousemove', event => { const rect = elements.progress.getBoundingClientRect(); - const percent = 100 / rect.width * (event.pageX - rect.left); + const percent = (100 / rect.width) * (event.pageX - rect.left); event.currentTarget.setAttribute('seek-value', percent); }); @@ -650,7 +645,7 @@ class Listeners { seek.removeAttribute('seek-value'); - player.currentTime = seekTo / seek.max * player.duration; + player.currentTime = (seekTo / seek.max) * player.duration; }, 'seek', ); |