Bug Tracker

Opened 6 years ago

Closed 6 years ago

#9100 closed bug (fixed)

Order of hide() callbacks has changed

Reported by: Odilon Redo Owned by: lrbabe
Priority: blocker Milestone: 1.6.1
Component: effects Version: 1.6
Keywords: Cc:
Blocked by: Blocking:

Description

The order in which callbacks for hide() are processed has reversed in 1.6 - previously (e.g. in 1.4,1.5) the callbacks would fire in DOM order (e.g. 1,2,3 in the code below) but they now fire in reverse (3,2,1). This behaviour is tested in Firefox 3.6.17 and Chrome 10.0.648.205.

<body>

<div id="div1">1</div>
<div id="div2">2</div>
<div id="div3">3</div>

<script src="http://code.jquery.com/jquery-1.6.min.js"></script>
<script>

	$("div").hide(1,function(){

		console.log($(this).attr("id"))


	});

</script>

</body>

Change History (5)

comment:1 Changed 6 years ago by addyosmani

Component: unfiledeffects
Priority: undecidedblocker
Status: newopen

Confirmed as a regression: http://jsfiddle.net/bEMRy/3/. As mentioned the order changes between 1.5.2 and 1.6.x with the issue still being present in our current edge.

comment:2 Changed 6 years ago by jaubourg

Milestone: 1.next1.6.1

comment:3 Changed 6 years ago by lrbabe

Owner: set to lrbabe
Status: openassigned

comment:5 Changed 6 years ago by louisremi

Resolution: fixed
Status: assignedclosed

Landing pull request 374. .animate() Callbacks should fire in correct order (unit test included). Fixes #9100.

More Details:

Note: See TracTickets for help on using tickets.