Opened 10 years ago
Closed 10 years ago
#13553 closed bug (fixed)
Leftover code for IE8 in effects.js
Reported by: | m_gol | Owned by: | |
---|---|---|---|
Priority: | undecided | Milestone: | None |
Component: | unfiled | Version: | git |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
In src/effects.js in current jQuery 2.0.0pre in lines 443-452 there is code:
// Remove in 2.0 - this supports IE8's panic based approach // to setting things on disconnected nodes Tween.propHooks.scrollTop = Tween.propHooks.scrollLeft = { set: function( tween ) { if ( tween.elem.nodeType && tween.elem.parentNode ) { tween.elem[ tween.prop ] = tween.now; } } };
I guess it should either be removed or the comment should be changed if it's still needed in 2.0.x.
Change History (2)
comment:2 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Edited comments and minor adjustments. Fixes #13553. Closes gh-1195.
Changeset: 79992d752393d80822a4f8be2b7bec3ecdd60f6a
Note: See
TracTickets for help on using
tickets.
See https://github.com/jquery/jquery/pull/1195
Turns out IE9 still needs this workaround. (fortunately, it seems IE10 doesn't)