« get me outta code hell

Add options to not call fixLayout automatically - tui-lib - Pure Node.js library for making visual command-line programs (ala vim, ncdu)
about summary refs log tree commit diff
path: root/ui/DisplayElement.js
diff options
context:
space:
mode:
authorFlorrie <towerofnix@gmail.com>2018-12-14 23:49:41 -0400
committerFlorrie <towerofnix@gmail.com>2018-12-14 23:49:41 -0400
commitd712d405f76178fc67b9421c113dccc3537e2a0c (patch)
treee43f2b49471fb6f0ef3bb9749d6d34207005dd50 /ui/DisplayElement.js
parent99d2ed5b2f8bb39fa2517efd5a68dbb2b27b3121 (diff)
Add options to not call fixLayout automatically
Diffstat (limited to 'ui/DisplayElement.js')
-rw-r--r--ui/DisplayElement.js14
1 files changed, 10 insertions, 4 deletions
diff --git a/ui/DisplayElement.js b/ui/DisplayElement.js
index 952c78e..8a2aa71 100644
--- a/ui/DisplayElement.js
+++ b/ui/DisplayElement.js
@@ -77,7 +77,7 @@ module.exports = class DisplayElement extends EventEmitter {
     }
   }
 
-  addChild(child, afterIndex = this.children.length) {
+  addChild(child, afterIndex = this.children.length, {fixLayout = true} = {}) {
     // TODO Don't let a direct ancestor of this be added as a child. Don't
     // let itself be one of its childs either!
 
@@ -88,10 +88,13 @@ module.exports = class DisplayElement extends EventEmitter {
 
     child.parent = this
     this.children.splice(afterIndex, 0, child)
-    child.fixLayout()
+
+    if (fixLayout) {
+      child.fixLayout()
+    }
   }
 
-  removeChild(child) {
+  removeChild(child, {fixLayout = true} = {}) {
     // Removes the given child element from the children list of this
     // element. It won't be rendered in the future. If the given element
     // isn't a direct child of this element, nothing will happen.
@@ -102,7 +105,10 @@ module.exports = class DisplayElement extends EventEmitter {
 
     child.parent = null
     this.children.splice(this.children.indexOf(child), 1)
-    this.fixLayout()
+
+    if (fixLayout) {
+      this.fixLayout()
+    }
   }
 
   centerInParent() {