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)

_autoComplete.html (10.7 KB) - added by bill 10 years ago.
simplified version of ComboBox? automated test, showing problem
8955.patch (6.3 KB) - added by Douglas Hays 10 years ago.
possible patch - please review for inclusion in 1.3

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by bill

Attachment: _autoComplete.html added

simplified version of ComboBox? automated test, showing problem

comment:1 Changed 10 years ago by bill

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 Douglas Hays

Milestone: tbd1.3
Priority: normalhigh
severity: normalmajor

IE 8 final changed a lot of things wrt scrolling

Changed 10 years ago by Douglas Hays

Attachment: 8955.patch added

possible patch - please review for inclusion in 1.3

comment:3 Changed 10 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

[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.

Note: See TracTickets for help on using tickets.