Opened 10 years ago
Closed 10 years ago
#8955 closed defect (fixed)
scrollIntoView problem scrolling up on IE8
| Reported by: | bill | Owned by: | Douglas Hays |
|---|---|---|---|
| Priority: | high | Milestone: | 1.3 |
| Component: | Dijit | Version: | 1.3.0b3 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
See attached version of tests/form/robot/_autoComplete.html.
It advances to the third page of results, then goes to the previous page, at which point the drop down is scrolled down to the bottom, so that the "next page" button is visible but "New Jersey" is scrolled out of view.
The scrollIntoView() call apparently does nothing. It should scroll the div upwards.
Attachments (2)
Change History (5)
Changed 10 years ago by
| Attachment: | _autoComplete.html added |
|---|
comment:1 Changed 10 years ago by
PS: to reproduce this, you need to make your browser window short enough that a scroll bar shows up on drop down list (which contains 30 items), and that only about 20 items are visible at one time.
comment:2 Changed 10 years ago by
| Milestone: | tbd → 1.3 |
|---|---|
| Priority: | normal → high |
| severity: | normal → major |
IE 8 final changed a lot of things wrt scrolling
Changed 10 years ago by
| Attachment: | 8955.patch added |
|---|
possible patch - please review for inclusion in 1.3
comment:3 Changed 10 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
[17094] Fixes #8955 !strict. Removed some IE8 specific code paths that are no longer needed by IE8 final. IE8+RTL mode also changed offsetLeft to be the physical left measurement (it was the offsetRight value in RC1) which is a complete reversal. I changed my internal offsetLeft to be the rc1/offsetright value to minimize code churn.
simplified version of ComboBox? automated test, showing problem