Bug Tracker

Ticket #1122 (closed bug: wontfix)

Opened 2 years ago

Last modified 9 months ago

Interface Slider onChange bug

Reported by: skimber Assigned to: stefan
Type: bug Priority: major
Milestone: 1.1.3 Component: interface
Version: 1.1.2 Keywords:
Cc: Needs: Review

Description (last modified by scott.gonzal) (diff)

With the Interface "Slider" plugin, clicking somewhere on the slider, rather than dragging the handle, does NOT cause the onChange function to be run.

I've tried altering the example script on interface.eyecon.ro... just changing the onSlide's to onChange's and adding alert('hello');

Same problem, so I'm fairly sure it's a bug and not just my code!

Attachments

Change History

Changed 2 years ago by nhoizey

Same error with jQuery 1.1.3 and Interface 1.2

Changed 1 year ago by kunzj

Its thelast comma in ui.slider.js: Remove the last comma on line 31:

change: function(e, ui) { cur[0].value = ui.value; cur[0].setAttribute("value", ui.value); },

in

change: function(e, ui) { cur[0].value = ui.value; cur[0].setAttribute("value", ui.value); }

Changed 9 months ago by scott.gonzal

  • status changed from new to closed
  • resolution set to wontfix
  • description changed from With the Interface "Slider" plugin, clicking somewhere on the slider, rather than dragging the handle, does NOT cause the onChange function to be run. I've tried altering the example script on interface.eyecon.ro... just changing the onSlide's to onChange's and adding alert('hello'); Same problem, so I'm fairly sure it's a bug and not just my code! to With the Interface "Slider" plugin, clicking somewhere on the slider, rather than dragging the handle, does NOT cause the onChange function to be run. I've tried altering the example script on interface.eyecon.ro... just changing the onSlide's to onChange's and adding alert('hello'); Same problem, so I'm fairly sure it's a bug and not just my code!

Interface is no longer supported; consider switching to jQuery UI.

Note: See TracTickets for help on using tickets.