Opened 6 years ago

Closed 4 years ago

#17151 closed defect (patchwelcome)

dojo/window::scrollIntoView scrolls down unnecessarily when body element shorter than content

Reported by: jeffreym Owned by:
Priority: undecided Milestone: 1.13
Component: HTML Version: 1.9.0
Keywords: Cc:
Blocked By: Blocking:

Description

When the <body> element is vertically smaller than the page content (because of e.g. an uncleared float or an absolutely positioned element), calling scrollIntoView on an element below the bottom of body will scroll down, even if the element is already completely in view.

Attachments (1)

scrollbug.html​ (691 bytes) - added by jeffreym 6 years ago.
An example - scroll to the top and click the yellow div; the window will scroll down even though the div is already in view, because it is below document.body.clientHeight

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by jeffreym

Attachment: scrollbug.html​ added

An example - scroll to the top and click the yellow div; the window will scroll down even though the div is already in view, because it is below document.body.clientHeight

comment:1 Changed 6 years ago by bill

Component: General → HTML
Owner: set to Douglas Hays
Status: new → assigned

Hmm, a DOMNode outside of the bounds of <body>, not sure if we want to support that but assigning to Doug.

comment:2 Changed 6 years ago by Douglas Hays

Owner: Douglas Hays deleted
Status: assigned → open

comment:3 Changed 4 years ago by dylan

Milestone: tbd → 1.12
Resolution: → patchwelcome
Status: open → closed

Given that no one has shown interest in creating a patch in the past 2+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.