Fix mousemove events for IE8

Fixes #829
flowplayer · Jun 4, 2015 · 2b232e6 · 2b232e6
1 parent 8d2257d
commit 2b232e6
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/lib/common.js b/lib/common.js
@@ -139,13 +139,15 @@ common.offset = function(el) {
 common.width = function(el, val) {
   /*jshint -W093 */
   if (val) return el.style.width = (''+val).replace(/px$/, '') + 'px';
-  return common.offset(el).width;
+  var ret = common.offset(el).width;
+  return typeof ret === 'undefined' ? el.offsetWidth : ret;
 };
 
 common.height = function(el, val) {
   /*jshint -W093 */
   if (val) return el.style.height = (''+val).replace(/px$/, '') + 'px';
-  return common.offset(el).height;
+  var ret = common.offset(el).height;
+  return typeof ret === 'undefined' ? el.offsetHeight : ret;
 };
 
 common.lastChild = function(el) {

diff --git a/lib/ext/slider.js b/lib/ext/slider.js
@@ -58,7 +58,7 @@ var slider = function(root, rtl) {
       },
 
       mousemove = function(e) {
-         var pageX = e.pageX;
+         var pageX = e.pageX || e.clientX;
          if (!pageX && e.originalEvent && e.originalEvent.touches && e.originalEvent.touches.length) {
             pageX = e.originalEvent.touches[0].pageX;
          }

diff --git a/lib/ext/ui.js b/lib/ext/ui.js
@@ -287,7 +287,7 @@ flowplayer(function(api, root) {
    });
 
    bean.on(root, 'mousemove', '.fp-timeline', function(ev) {
-     var x = ev.pageX,
+     var x = ev.pageX || ev.clientX,
          delta = x - common.offset(timeline).left,
          percentage = delta / common.width(timeline),
          seconds = percentage * api.video.duration;