#5178 closed bug (fixed)

Opened February 13, 2010 01:07AM UTC

Closed February 15, 2010 02:35PM UTC

Slider's change event fires on slide

Reported by: vmx Owned by:
Priority: blocker Milestone: 1.8
Component: ui.slider Version: 1.8rc1
Keywords: Cc:
Blocked by: Blocking:
Description

When you attach a callback to the sliders "change" event (.slider({change: function() {/*idosomething*/}});) it is called on every move of the slider, just like the "slide" event. It's not the expected behaviour that it only fires when the value changed.

It's a regression, it works in UI 1.7.2.

To see the problem, replace the "slide" callback in the steps demo with "change".

Attachments (1)
  • ui-slider-change.patch (1.1 KB) - added by vmx February 13, 2010 01:45AM UTC.

    Don't fire change event on slide

Change History (4)

Changed February 13, 2010 01:47AM UTC by vmx comment:1

Please have a look at the patch. Though I'm not really happy with it, as it is more of a hack than a fix.

Changed February 14, 2010 04:51AM UTC by scottgonzalez comment:2

priority: critical → blocker

Changed February 15, 2010 02:13PM UTC by rdworth comment:3

regression occurred is r3361

Changed February 15, 2010 02:35PM UTC by rdworth comment:4

resolution: → fixed
status: new → closed

Fixed in r3800