Opened 15 years ago

Closed 15 years ago

Last modified 14 years ago

#2366 closed bug (fixed)

slider with two handles: When stepping < 1/4 of realMaxValue ->wrong steps created

Reported by: nettoyeur Owned by: eduardo
Priority: major Milestone: 1.5
Component: ui.core Version: 1.2.3
Keywords: slider, stepping, jqueryui Cc:
Blocked by: Blocking:

Description

ui.slider.js, r4730

When minValue=0 and stepping is smaller then 1/4 of maxvalue, it fails to create the proper steps (rounding error?)

how to reproduce: Create slider with two handles.

minvalue:0
maxvalue:1000
stepping:250

->everything ok, steps of 250 are taken

Now try smaller stepping:

minvalue:0
maxvalue:1000
stepping:200

->first step with left slider is 196.8085106382979, second step is 398.93617021276594, third step is 595.7446808510638, fourth step is 797.8723404255319, fifth step is 1000. (expected: 200,400,600,800,1000)

Attachments (1)

ui_slider_value_round_on_stepping.diff (1.2 KB) - added by zimbatm 15 years ago.
Round value() if stepping

Download all attachments as: .zip

Change History (9)

comment:1 Changed 15 years ago by paul

Owner: changed from paul to braeker

comment:2 Changed 15 years ago by tfe

This also occurs in single-handle sliders.

comment:3 Changed 15 years ago by tfe

Also, I could really use a fix for this issue ASAP. I need to use 1.5b to be able to use a slider with multiple handles, but using 1.5b also breaks my sliders with this bug.

comment:4 Changed 15 years ago by zimbatm

This is because .value() calculates itself from the position of the element instead of the current position on the stepping

Changed 15 years ago by zimbatm

Round value() if stepping

comment:5 Changed 15 years ago by zimbatm

Please apply the patch. tests are passing

comment:6 Changed 15 years ago by Scott González

Resolution: fixed
Status: newclosed

Fixed in [4963].

comment:7 Changed 15 years ago by (none)

Milestone: 1.2.4

Milestone 1.2.4 deleted

comment:8 Changed 14 years ago by paul

Milestone: 1.5
Note: See TracTickets for help on using tickets.