#3983 closed defect (fixed)
Slider: with no discreteValues fails to send events via buttons
Reported by: | guest | Owned by: | Douglas Hays |
---|---|---|---|
Priority: | high | Milestone: | 1.0 |
Component: | Dijit - Form | Version: | 0.9 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
HOW TO REPRODUCE:
Go to <http://82.230.65.68:31416/~fred/dojo/fred_samples/bug_002.html>
Click +
EXPECTED RESULT:
Value change to something like 0.0009852216748768472
RESULT:
Event is not fired, value not updated
Attachments (1)
Change History (13)
Changed 14 years ago by
Attachment: | bug_002.html added |
---|
comment:2 Changed 14 years ago by
Owner: | changed from bill to Douglas Hays |
---|
comment:3 Changed 14 years ago by
Milestone: | → 0.9 |
---|
comment:4 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:5 Changed 14 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
I must be very stupid, but, with revision [10028], I still have the reported behavior.
Click on link bug_002.html Click on + button several time Value stays zero (If you click anywhere in the window, it is updated, but it is too late)
I also think the title of the issue is wrong: it should be "Slider with no intermediateChanges" don't send events via buttons.
comment:6 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Clicking the buttons is an intermediate change - since intermediateChanges=false, no onChange event fires. Clicking the draggable handle and using the arrow keys are also intermediate changes. When the widget loses focus, then the onChange event fires for the last intermediate change. Dragging the handle is also an intermediate change, but the mouseup event after dragging causes the onChange event to fire to allow webkit users who can't focus on div's to use the slider.
comment:7 Changed 14 years ago by
Omg, I did not realise that this was the expected behavior. Time to argue against that on the forums, then.
Thanks for the update.
comment:8 Changed 14 years ago by
Milestone: | 0.9 → 1.0 |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
Summary: | dijit.form.HorizontalSlider with no discreteValues fails to send events via buttons → Slider: with no discreteValues fails to send events via buttons |
OK, we talked about this on IRC for a while. Decided it was too confusing to not fire an onChange event on arrow key or arrow button (imagine that a vertical slider is used to control the position in a table widget and the user hits the + button to go down one page). So let's change this to fire on arrow keyup or arrow button.
comment:9 Changed 14 years ago by
The suggested behavior is different than the Spinner widget which is only slightly different the slider. Also, we might want slider to behave well with InlineEditBox? - and an immediate value change will close the editing sesson.
comment:11 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
comment:12 Changed 10 years ago by
Component: | Dijit → Dijit - Form |
---|
Test case, as linked in the report