Side navigation
#13553 closed bug (fixed)
Opened March 02, 2013 11:22AM UTC
Closed March 13, 2013 01:08AM UTC
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.
Attachments (0)
Change History (2)
Changed March 04, 2013 05:24AM UTC by comment:1
_comment0: | See https://github.com/jquery/jquery/pull/1195 \ \ Turns out IE9 still needs this workaround. → 1362374679919169 |
---|
See https://github.com/jquery/jquery/pull/1195
Turns out IE9 still needs this workaround. (fortunately, it seems IE10 doesn't)