Changes between Version 2 and Version 3 of Ticket #18726


Ignore:
Timestamp:
Oct 5, 2015, 12:56:21 PM (6 years ago)
Author:
bill
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #18726

    • Property Summary changed from [regression] scrollIntoView() broken for IE8 quirks and loose mode to [regression] scrollIntoView() broken for IE8 quirks RTL
  • Ticket #18726 – Description

    v2 v3  
    1 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.
     1The changes from https://github.com/dojo/dojo/commit/a832214a342b6e03ad4a9b1669fae71c7938c44b broke `scrollIntoView()` on IE8 for quirks RTL, and then masked the error by inadvertently disabling those tests on IE8.
    22
    33The mistake was because Mark was thinking the `has("trident")` is just for IE11+, but it's actually truthy for IE8 - IE11.