#17297 closed defect (fixed)
RangeSlider: clicking remaining bar yields nonsense values
Reported by: | Douglas Hays | Owned by: | Colin Snover |
---|---|---|---|
Priority: | undecided | Milestone: | 1.7.6 |
Component: | DojoX Form | Version: | 1.9.1 |
Keywords: | Cc: | bill | |
Blocked By: | Blocking: |
Description
When you click the remaining bar (outside the current handles but on the bar), the relative mouse position is compared to the absolute bar position which yields nonsense values.
Change History (12)
comment:1 Changed 8 years ago by
Component: | General → DojoX Form |
---|---|
Milestone: | tbd → 1.10 |
Owner: | set to Douglas Hays |
Status: | new → assigned |
comment:2 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:3 Changed 7 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
This needs to be backported.
comment:4 Changed 7 years ago by
Cc: | bill added |
---|---|
Owner: | Douglas Hays deleted |
Status: | reopened → assigned |
doughays won't backport this. So csnover and/or wkeese I guess if you want this to be backported you should do it otherwise we should close this.
comment:5 Changed 7 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
I tried 1.9 and 1.8, and clicking the bar seems to work fine to me. So I have no way to test a backport, even if I wanted to do it.
comment:6 Changed 7 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
It’s easy to reproduce in both Firefox and Chrome.
- Open dojox/form/tests/test_RangeSlider.html
- Click in the white space below the second slider under Vertical slider, or the white space to the right of the second slider under Horizontal slider
Expected: Bottom/right slider moves to the clicked point Actual: Top/left slider moves to the clicked point
comment:7 Changed 7 years ago by
Owner: | set to Colin Snover |
---|---|
Status: | reopened → assigned |
I see. I'm not going to backport because the cherry-pick doesn't apply cleanly, since _RangeSliderMixin.js was deleted after the change. Feel free to backport if you like.
comment:8 Changed 7 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:11 Changed 7 years ago by
Milestone: | 1.10 → 1.7.6 |
---|
In f21df6fbd6063ca4295d8932d12ffa463a36a365/dojox: