Merge pull request mediaelement#335 from jmcneese/volume-slider-offse…

…t-fix added check for left offset to detect when mousedrag exceeds top boundar...
rexblack · Dec 28, 2011 · a7b23fa · a7b23fa
2 parents 31d0648 + 4d9a319
commit a7b23fa
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/build/mediaelement-and-player.js b/build/mediaelement-and-player.js
@@ -2916,7 +2916,7 @@ if (typeof jQuery != 'undefined') {
 					volume = (railHeight - newY) / railHeight
 
 				// the controls just hide themselves (usually when mouse moves too far up)
-				if (totalOffset.top == 0)
+				if (totalOffset.top == 0 || totalOffset.left == 0)
 					return;
 
 				// 0-1

diff --git a/build/mediaelement-and-player.min.js b/build/mediaelement-and-player.min.js
diff --git a/build/mediaelementplayer.js b/build/mediaelementplayer.js
@@ -1437,7 +1437,7 @@ if (typeof jQuery != 'undefined') {
 					volume = (railHeight - newY) / railHeight
 
 				// the controls just hide themselves (usually when mouse moves too far up)
-				if (totalOffset.top == 0)
+				if (totalOffset.top == 0 || totalOffset.left == 0)
 					return;
 
 				// 0-1

diff --git a/build/mediaelementplayer.min.js b/build/mediaelementplayer.min.js
diff --git a/src/js/mep-feature-volume.js b/src/js/mep-feature-volume.js
@@ -65,7 +65,7 @@
 					volume = (railHeight - newY) / railHeight
 
 				// the controls just hide themselves (usually when mouse moves too far up)
-				if (totalOffset.top == 0)
+				if (totalOffset.top == 0 || totalOffset.left == 0)
 					return;
 
 				// 0-1