Side navigation
#9072 closed bug (fixed)
Opened May 03, 2011 06:30PM UTC
Closed May 03, 2011 07:03PM UTC
Last modified March 17, 2012 08:43PM UTC
jQuery 1.6 Crashes IE 6
| Reported by: | john | Owned by: | |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.6.1 |
| Component: | manipulation | Version: | 1.6 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
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
Note the inner loop is clobbering the outer loop's "i" variable.
From here: http://news.ycombinator.com/item?id=2510844
Attachments (0)
Change History (6)
Changed May 03, 2011 06:31PM UTC by comment:1
| description: | 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. \ \ From here: http://news.ycombinator.com/item?id=2510844 → 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 \ \ Note the inner loop is clobbering the outer loop's "i" variable. \ \ From here: http://news.ycombinator.com/item?id=2510844 |
|---|
Changed May 03, 2011 06:34PM UTC by comment:2
| component: | unfiled → manipulation |
|---|---|
| milestone: | 1.next → 1.6.1 |
| priority: | undecided → blocker |
Changed May 03, 2011 07:03PM UTC by comment:3
| resolution: | → fixed |
|---|---|
| status: | new → closed |
Fix problem with appending multiple string arguments in IE 6. Fixes #9072.
Changeset: bfad45fe4510db78a38170752ccc9c9efe5f38b6
Changed May 14, 2011 09:26AM UTC by comment:6
A heads up: I see this was fixed in jQuery 1.6.1 but http://ajax.googleapis.com/ajax/libs/jquery/1/jquery.js, which I thought was the latest jQuery, has the bug in it. I wasted hours debugging today because of this.