5 ms·
Simple reproduce case: <html> <head> <script src="http://code.jquery.com/jquery-1.6.min.js"></script> </head> <body> <script> $('body').append(
by 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 agoFantastic 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 agoOof! 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 agoThe 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 agoHe gets a reward! http://rewardjs.com/ http://rewardjs.com/
- rwaldron 15y agoIn 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.