Merge pull request #166 from aimeos/dev

phphe · web-flow · commit 81e61c2cb406 · 2026-04-13T10:39:33.000+08:00
Fixed off-by-one when moving nodes by keyboard
diff --git a/packages/he-tree-vue/src/components/DraggableTree.ts b/packages/he-tree-vue/src/components/DraggableTree.ts
@@ -229,7 +229,7 @@ const cpt = defineComponent({
           }
           case "ArrowDown": {
             if (siblingIndex < siblings.length - 1) {
-              this.move(stat, stat.parent || null, siblingIndex + 2);
+              this.move(stat, stat.parent || null, siblingIndex + 1);
               this.$emit("change");
               const newSiblings = stat.parent
                 ? stat.parent.children

Original file line number	Diff line number	Diff line change
`@@ -229,7 +229,7 @@ const cpt = defineComponent({`
`229`	`229`	`}`
`230`	`230`	`case "ArrowDown": {`
`231`	`231`	`if (siblingIndex < siblings.length - 1) {`
`232`		`- this.move(stat, stat.parent \|\| null, siblingIndex + 2);`
	`232`	`+ this.move(stat, stat.parent \|\| null, siblingIndex + 1);`
`233`	`233`	`this.$emit("change");`
`234`	`234`	`const newSiblings = stat.parent`
`235`	`235`	`? stat.parent.children`