Hacker News Reader

Comment by wyday | Hacker News Reader

This release causes Internet Explorer 6 to freeze just by including the script in the page. This is a non-starter. I'll submit a bug report about it later today, just thought you guys would like to know in the meantime.

EDIT: It looks like the freeze on IE6 might be caused by FancyBox (http://fancybox.net/ ). Though it didn't freeze with any previous version of jQuery.

View on HN

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.

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.

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/

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.

Reply on news.ycombinator.com