From 7e1e2b96ffab68ecccb010e99ef38296649e057b Mon Sep 17 00:00:00 2001 From: MattMo Date: Tue, 31 Oct 2023 22:45:02 -0700 Subject: [PATCH] Fixed a minor bug where list splice wouldn't insert the item in the DOM at the correct position in the list. --- ignite-template.js | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/ignite-template.js b/ignite-template.js index 25b76c9..39baf58 100644 --- a/ignite-template.js +++ b/ignite-template.js @@ -2523,7 +2523,12 @@ class list extends IgniteTemplate { items.forEach(item => { var template = this.forEachCallback(item, start); - template.construct(this.element.parentElement, this.element); + //If we have elements already, construct this item after the element at the current start. Otherwise use the placeholder element. + if (this.elements.length > 0) { + template.construct(this.element.parentElement, this.elements[start]); + } else { + template.construct(this.element.parentElement, this.element); + } if (this.reflecting) { this.reflectCallbacks.splice(start, 0, template.element.attachOnDisconnect(disconnect => this.onItemRemove(item)));