aboutsummaryrefslogtreecommitdiffstats
path: root/src/js/listeners.js
diff options
context:
space:
mode:
authorSam Potts <sam@potts.es>2018-09-25 20:43:09 +1000
committerSam Potts <sam@potts.es>2018-09-25 20:43:09 +1000
commit62d80e6b7698d200d3b97c6084f059e2ba780efe (patch)
treefd8ef2fa03faad84fde0a89c38fd669480842f68 /src/js/listeners.js
parent7dc4d9cd22a49ba4019671cf703b25b162bcfa19 (diff)
downloadplyr-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.js19
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',
);