MontoyaTech/ignite-html
MontoyaTech/ignite-html
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fixed a minor bug where list splice wouldn't insert the item in the DOM at the correct position in the list.

Browse Source
This commit is contained in:
MattMo 2023-10-31 22:45:02 -07:00
parent a584eb68a7
commit 7e1e2b96ff
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ignite-template.js
View File

@ -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)));