Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

A reduced bug report would be good. We haven't seen any reports of this elsewhere (and the jQuery test suite runs to completion in IE 6) so I'm not entirely sure what could be happening here. Any insight you can provide (and a bug report) would be appreciated.


Simple reproduce case:

  <html>
  <head>
  <script src="http://code.jquery.com/jquery-1.6.min.js"></script>
  </head>
  <body>
  <script>
  $('body').append(
    '<span></span>',
    '<span></span>',
    '<span></span>'
  )
  </script>
  </body>
  </html>
The bug is in this block: https://github.com/jquery/jquery/blob/1.6/src/manipulation.j...

Note the inner loop is clobbering the outer loop's "i" variable.


Oof! That's no good, at all. We'll fix this right away. Thanks for the reduction. Just filed it here: http://bugs.jquery.com/ticket/9072


The issue has been fixed and integrated into the current build of jQuery: http://code.jquery.com/jquery-git.js

We're going to have a 1.6.1 release within the week. Thank you for your bug report!


He gets a reward!

http://rewardjs.com/


Fantastic find, crescentfresh! Changing "i" to "j" in the inner loop fixes the problem on IE6.

Now we just have to wait for jQuery 1.6.1.


In the future, please post any jQuery bugs you happen to find in the jQuery Bug Tracker http://bugs.jquery.com/

This helps us to help you! And remember, the future in San Dimas is excellent... so be excellent to each other.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: