Stop using pointers for button labels

js/term/buttons.js

@@ -60,5 +60,14 @@ module.exports = function initButtons (input) {
     }
   }
 
-  return { update, labels }
+  return {
+    update,
+    get labels () {
+      return labels
+    },
+    set labels (value) {
+      labels = value
+      update()
+    }
+  }
 }

js/term/connection.js

@@ -145,7 +145,6 @@ module.exports = class TermConnection extends EventEmitter {
       return true // Simulate success
     }
     if (this.xoff) {
-      // TODO queue
       console.log("Can't send, flood control. Queueing")
       this.queue.push(message)
       return false

js/term/index.js

@@ -20,11 +20,7 @@ module.exports = function (opts) {
   input.termUpload = termUpload
 
   const buttons = initButtons(input)
-  screen.on('button-labels', labels => {
+  screen.on('button-labels', labels => { buttons.labels = labels })
-    // TODO: don't use pointers for this
-    buttons.labels.splice(0, buttons.labels.length, ...labels)
-    buttons.update()
-  })
 
   screen.on('TEMP:hide-load-failed-msg', () => {
     let scr = qs('#screen')