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)