Side navigation
#10177 closed bug (fixed)
Opened August 31, 2011 11:50AM UTC
Closed November 06, 2011 10:03PM UTC
Last modified November 06, 2011 10:04PM UTC
index of callback function in .wrap is always 0
Reported by: | manuel@bouza.ch | Owned by: | rwaldron |
---|---|---|---|
Priority: | high | Milestone: | 1.7.1 |
Component: | manipulation | Version: | 1.6.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
I have the following HTML:
<div class="box">...</div>
<div class="box">...</div>
Using the following:
$J('div.box').wrap(function(index) { return '<div class="boxwrap index_' + index +'" />' });
I expect to get:
<div class="boxwrap index_0"> <div class="box">...</div> </div> <div class="boxwrap index_1"> <div class="box">...</div> </div>
Error, I get always 0 for index:
<div class="boxwrap index_0"> <div class="box">...</div> </div> <div class="boxwrap index_0"> <div class="box">...</div> </div>
The follwing workaround produces the expected result:
$J('div.box').each(function(index) { $J(this).wrap(function() { return '<div class="boxwrap index_' + index +'" />'; }); });
Attachments (0)
Change History (5)
Changed August 31, 2011 02:13PM UTC by comment:1
component: | unfiled → manipulation |
---|---|
owner: | → manuel@bouza.ch |
priority: | undecided → low |
status: | new → pending |
Changed September 01, 2011 07:29AM UTC by comment:2
status: | pending → new |
---|
Here is the link to the test using jQuery edge in jsFiddle: http://jsfiddle.net/8SZAx/2/ showing the bug.
Here is the link to the work around with the expected result: http://jsfiddle.net/LBbrn/1/
Changed September 01, 2011 02:39PM UTC by comment:3
milestone: | None → 1.next |
---|---|
owner: | manuel@bouza.ch → rwaldron |
priority: | low → high |
status: | new → assigned |
Confirmed: http://jsfiddle.net/rwaldron/FjXLB/
Changed November 06, 2011 10:03PM UTC by comment:4
resolution: | → fixed |
---|---|
status: | assigned → closed |
Fix #10177. Pass correct index to function-parameter of .wrap
Changeset: 47c605f693457412a53c9145cafe55420ab8f236
Changed November 06, 2011 10:04PM UTC by comment:5
milestone: | 1.next → 1.7.1 |
---|
Thanks for taking the time to contribute to the jQuery project! Please provide a reduced test case on jsFiddle to help us assess your ticket.
Additionally, be sure to test against the jQuery Edge version to ensure the issue still exists. To get you started, use this boilerplate: http://jsfiddle.net/FrKyN/ Open the link and click to "Fork" (in the top menu) to get started.