Bug Tracker

Modify

Ticket #9100 (closed bug: fixed)

Opened 3 years ago

Last modified 3 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 3 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 3 years ago by jaubourg

  • Milestone changed from 1.next to 1.6.1

comment:3 Changed 3 years ago by lrbabe

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

comment:4 Changed 3 years ago by lrbabe

comment:5 Changed 3 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:

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.