Skip to main content

Bug Tracker

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 m_gol 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)

Changed March 13, 2013 01:08AM UTC by Michał Z. Gołębiowski comment:2

resolution: → fixed
status: newclosed

Edited comments and minor adjustments. Fixes #13553. Closes gh-1195.

Changeset: 79992d752393d80822a4f8be2b7bec3ecdd60f6a