Bug Tracker

Ticket #9100 (closed bug: fixed)

Opened 4 years ago

Last modified 4 years ago

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:
Blocking: Blocked by:

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

comment:1 Changed 4 years ago by addyosmani

  • Priority changed from undecided to blocker
  • Status changed from new to open
  • Component changed from unfiled to effects

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 4 years ago by jaubourg

  • Milestone changed from 1.next to 1.6.1

comment:3 Changed 4 years ago by lrbabe

  • Owner set to lrbabe
  • Status changed from open to assigned

comment:4 Changed 4 years ago by lrbabe

comment:5 Changed 4 years ago by louisremi

  • Status changed from assigned to closed
  • Resolution set to fixed

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.