IE10+ fail the test for modifying offset values in a callback #2147

Closed
mgol opened this Issue Mar 16, 2015 · 0 comments

Comments

Projects
None yet
2 participants

@mgol mgol added this to the 3.0.0 milestone Mar 16, 2015

@mgol mgol self-assigned this Mar 16, 2015

@mgol mgol added the Offset label Mar 16, 2015

mgol added a commit to mgol/jquery that referenced this issue Mar 16, 2015

Offset: Round offset value for the sake of floating errors
IE10+ may return not exactly the offset.top value set in an offset callback
if parent has fractional top offset itself. Checking for being close to the
desired result Fixes the test error.

Fixes gh-2147

mgol added a commit to mgol/jquery that referenced this issue Mar 16, 2015

Offset: Round offset value for the sake of floating errors
IE10+ may return not exactly the offset.top value set in an offset callback
if parent has fractional top offset itself. Checking for being close to the
desired result Fixes the test error.

Fixes gh-2147

@mgol mgol closed this in 62ae2d0 Mar 23, 2015

mgol added a commit that referenced this issue Mar 23, 2015

Offset: Round offset value for the sake of floating errors
IE10+ may return not exactly the offset.top value set in an offset callback
if parent has fractional top offset itself. Checking for being close to the
desired result fixes the test error.

(cherry-picked from 62ae2d0)

Fixes gh-2147

mgol added a commit that referenced this issue Nov 10, 2015

Offset: Round offset value for the sake of floating errors
IE10+ may return not exactly the offset.top value set in an offset callback
if parent has fractional top offset itself. Checking for being close to the
desired result Fixes the test error.

Fixes gh-2147

@dmethvin dmethvin modified the milestones: 1.12/2.2, 3.0.0 Jan 7, 2016

@cssmagic cssmagic referenced this issue in cssmagic/ChangeLog May 18, 2016

Open

jQuery #5

@jquery jquery locked as resolved and limited conversation to collaborators Jun 19, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.