diff options
author | Florrie <towerofnix@gmail.com> | 2018-12-15 01:31:52 -0400 |
---|---|---|
committer | Florrie <towerofnix@gmail.com> | 2018-12-15 01:31:52 -0400 |
commit | e47a3546f0f9244bb6ee3aa708e7e24226fbdad2 (patch) | |
tree | 103b3f94287a2c937bc0dc833982dfe3b0e7873e | |
parent | d54498610d53e8c8a437a6adff0ced11b037afe7 (diff) |
Fix mistake in Form.addInput
TL;DR afterIndex was not being set correctly.
-rw-r--r-- | ui/DisplayElement.js | 7 | ||||
-rw-r--r-- | ui/form/Form.js | 2 |
2 files changed, 7 insertions, 2 deletions
diff --git a/ui/DisplayElement.js b/ui/DisplayElement.js index 8a2aa71..9498487 100644 --- a/ui/DisplayElement.js +++ b/ui/DisplayElement.js @@ -87,7 +87,12 @@ module.exports = class DisplayElement extends EventEmitter { } child.parent = this - this.children.splice(afterIndex, 0, child) + + if (afterIndex === this.children.length) { + this.children.push(child) + } else { + this.children.splice(afterIndex, 0, child) + } if (fixLayout) { child.fixLayout() diff --git a/ui/form/Form.js b/ui/form/Form.js index 7a82b31..3c59cf5 100644 --- a/ui/form/Form.js +++ b/ui/form/Form.js @@ -19,7 +19,7 @@ module.exports = class Form extends FocusElement { this.inputs.push(input) if (asChild) { - this.addChild(input, opts) + this.addChild(input, this.children.length, opts) } } |