Opened 6 years ago

Last modified 6 years ago

#18726 closed defect

[regression] scrollIntoView() broken for IE8 quirks and loose mode — at Version 2

Reported by: bill Owned by: bill
Priority: undecided Milestone: 1.7.9
Component: Core Version: 1.7.7
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

The changes from https://github.com/dojo/dojo/commit/a832214a342b6e03ad4a9b1669fae71c7938c44b broke scrollIntoView() on IE8 for quirks and loose mode, and then masked the error by inadvertently disabling those tests on IE8.

The mistake was because Mark was thinking the has("trident") is just for IE11+, but it's actually truthy for IE8 - IE11.

Change History (2)

comment:1 Changed 6 years ago by bill

Description: modified (diff)

comment:2 Changed 6 years ago by bill

Description: modified (diff)
Milestone: tbd1.7.9
Owner: set to bill
Status: newassigned
Summary: scrollIntoView() suspicious code for IE8[regression] scrollIntoView() broken for IE8 quirks and loose mode
Version: 1.10.41.7.7
Note: See TracTickets for help on using tickets.