Minor bugfix - "currentSplitter not set" when clicking handle on a FI…

…XED splitter.
hiltonjanfield · Oct 8, 2015 · 848e048 · 848e048
1 parent 7751873
commit 848e048
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/js/jquery.enhsplitter.js b/js/jquery.enhsplitter.js
@@ -322,11 +322,14 @@
 
                 .on('mousedown.splitter', '.splitter_handle', function (e) {
                     e.preventDefault();
-                    if (currentSplitter == null) {
+                    if (currentSplitter === null) {
                         $(this).closest('.splitter_bar').trigger('mousedown');
                     }
 
-                    dragStartPosition = (currentSplitter.settings.vertical) ? e.pageX : e.pageY;
+                    // Two separate comparisons on purpose. .trigger() may or may not set currentSplitter.
+                    if (currentSplitter !== null) {
+                        dragStartPosition = (currentSplitter.settings.vertical) ? e.pageX : e.pageY;
+                    }
                 })
 
                 .on('mousedown.splitter touchstart.splitter', '.splitter_container > .splitter_bar', function (e) {