Skip to content
Permalink
Browse files

slider: fixed regression caused by a recent change in the mouse core …

…- since the mouse core doesn't stop mousedown propagation by default now, the mousedown event was triggered twice - once by the bound handle, and once by the actual element.
  • Loading branch information...
Paul Bakaus
Paul Bakaus committed Nov 24, 2008
1 parent 0ef9a8a commit 2b92a02170babdf2d970ecabd2a326a0a7d64f5b
Showing with 11 additions and 2 deletions.
  1. +11 −2 ui/ui.slider.js
@@ -72,9 +72,18 @@ $.widget("ui.slider", {

// Bind the click to the slider itself
this.element.bind('mousedown.slider', function(event) {

if($(event.target).is('.ui-slider-handle')) return;

//Go to the actual clicked posiion, apply a click
self._click.apply(self, [event]);
self.currentHandle.data("mouse").trigger(event);
self.firstValue = self.firstValue + 1; //This is for always triggering the change event

//initiate a handle drag, so we can click+drag somewhere
self.currentHandle.data("mouse").trigger(event);

//This is for always triggering the change event
self.firstValue = self.firstValue + 1;

});

// Move the first handle to the startValue

0 comments on commit 2b92a02

Please sign in to comment.
You can’t perform that action at this time.