9 ms·

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

by wyday 15y ago

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/ http://fancybox.net/ ). Though it didn't freeze with any previous version of jQuery.

deleted 15y ago
[deleted]
jeresig 15y ago
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.
- deleted 15y ago
  [deleted]
- crescentfresh 15y ago
  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.js#L652 https://github.com/jquery/jquery/blob/1.6/src/manipulation.j... Note the inner loop is clobbering the outer loop's "i" variable.
  - wyday 15y ago
    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.
  - jeresig 15y ago
    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 http://bugs.jquery.com/ticket/9072
    - jeresig 15y ago
      The issue has been fixed and integrated into the current build of jQuery: http://code.jquery.com/jquery-git.js 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!
      - DTrejo 15y ago
        He gets a reward! http://rewardjs.com/ http://rewardjs.com/
  - rwaldron 15y ago
    In the future, please post any jQuery bugs you happen to find in the jQuery Bug Tracker http://bugs.jquery.com/ 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.

by wyday 15y ago

deleted 15y ago

[deleted]

jeresig 15y ago

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.

deleted 15y ago
[deleted]
crescentfresh 15y ago
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.js#L652 https://github.com/jquery/jquery/blob/1.6/src/manipulation.j... Note the inner loop is clobbering the outer loop's "i" variable.
- wyday 15y ago
  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.
- jeresig 15y ago
  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 http://bugs.jquery.com/ticket/9072
  - jeresig 15y ago
    The issue has been fixed and integrated into the current build of jQuery: http://code.jquery.com/jquery-git.js 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!
    - DTrejo 15y ago
      He gets a reward! http://rewardjs.com/ http://rewardjs.com/
- rwaldron 15y ago
  In the future, please post any jQuery bugs you happen to find in the jQuery Bug Tracker http://bugs.jquery.com/ 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.