fix(template-strategy): move views correctly

martingust · martingust · commit 5b5e6e6d4e70 · 2016-06-27T20:34:18.000+02:00
Fixes #68
diff --git a/src/template-strategy.js b/src/template-strategy.js
@@ -41,11 +41,13 @@ export class TableStrategy {
   }
 
   moveViewFirst(view: View, topBuffer: Element): void {
-    insertBeforeNode(view, DOM.nextElementSibling(topBuffer.parentNode).previousSibling);
+    insertBeforeNode(view, DOM.nextElementSibling(topBuffer.parentNode));
   }
 
   moveViewLast(view: View, bottomBuffer: Element): void {
-    insertBeforeNode(view, bottomBuffer.parentNode);
+    let previousSibling = bottomBuffer.parentNode.previousSibling;
+    let referenceNode = previousSibling.nodeType === 8 && previousSibling.data === 'anchor' ? previousSibling : bottomBuffer.parentNode;
+    insertBeforeNode(view, referenceNode);
   }
 
   createTopBufferElement(element: Element): Element {

Original file line number	Diff line number	Diff line change
`@@ -41,11 +41,13 @@ export class TableStrategy {`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`moveViewFirst(view: View, topBuffer: Element): void {`
`44`		`- insertBeforeNode(view, DOM.nextElementSibling(topBuffer.parentNode).previousSibling);`
	`44`	`+ insertBeforeNode(view, DOM.nextElementSibling(topBuffer.parentNode));`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`moveViewLast(view: View, bottomBuffer: Element): void {`
`48`		`- insertBeforeNode(view, bottomBuffer.parentNode);`
	`48`	`+ let previousSibling = bottomBuffer.parentNode.previousSibling;`
	`49`	`+ let referenceNode = previousSibling.nodeType === 8 && previousSibling.data === 'anchor' ? previousSibling : bottomBuffer.parentNode;`
	`50`	`+ insertBeforeNode(view, referenceNode);`
`49`	`51`	`}`
`50`	`52`
`51`	`53`	`createTopBufferElement(element: Element): Element {`