Closed Bug 289547 Opened 20 years ago Closed 20 years ago

setindex does not work properly for nested repeats

Categories

(Core Graveyard :: XForms, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: allan, Assigned: allan)

References

()

Details

Attachments

(2 files, 1 obsolete file)

6.57 KB, application/xhtml+xml
Details
1023 bytes, patch
smaug
: review+
aaronr
: review+
Details | Diff | Splinter Review
 
Attached file Testcase (obsolete) —
Testcase with three levels of repeats and setindex for them. The middle repeat
does weird stuff and the inner repeat does nothing. *sigh*
Attached file Testcase v2
Updated the testcase to use class for styling instead of id.
Attachment #180044 - Attachment is obsolete: true
Attached patch PatchSplinter Review
Ok, that was simple :) This patch removes the id attribute on cloned repeat
content (as I should have done a long time ago). This solves the problem, as it
is now the DOM repeat handling the setindex call, as it should have been all
the time.
Attachment #182160 - Flags: review?(smaug)
Status: NEW → ASSIGNED
Attachment #182159 - Attachment is patch: false
Attachment #182159 - Attachment mime type: text/plain → application/xhtml+xml
Attachment #182160 - Flags: review?(smaug) → review+
Attachment #182160 - Flags: review?(aaronr)
Attachment #182160 - Flags: review?(aaronr) → review+
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: