use common modal methods for visibility

Author: jjasonclark

lib/dashboard.js

@@ -72,21 +72,21 @@ Dashboard.prototype._configureKeys = function () {
     this._showLayout(target);
   }.bind(this), THROTTLE_TIMEOUT));
 
-  var helpNode = this.helpView.node;
   this.container.key(["?", "h", "S-h"], function () {
-    helpNode.toggle();
+    this.gotoTimeView.hide();
+    this.helpView.toggle();
     this.screen.render();
   }.bind(this));
 
   this.container.key(["g", "S-g"], function () {
-    helpNode.hide();
+    this.helpView.hide();
     this.gotoTimeView.toggle();
     this.screen.render();
   }.bind(this));
 
   this.container.key("escape", function () {
-    if (helpNode.visible || this.gotoTimeView.isVisible()) {
-      helpNode.hide();
+    if (this.helpView.isVisible() || this.gotoTimeView.isVisible()) {
+      this.helpView.hide();
       this.gotoTimeView.hide();
       this.screen.render();
     } else {

lib/views/help.js

@@ -43,7 +43,45 @@ var HelpView = function HelpView(options) {
     hidden: true
   });
 
+  this.node.on("show", function () {
+    options.parent.screen.saveFocus();
+    this.node.setFront();
+  }.bind(this));
+
+  this.node.on("hide", function () {
+    options.parent.screen.restoreFocus();
+  });
+
   options.parent.append(this.node);
 };
 
+/**
+ * Toggle the visibility of the view.
+ *
+ * @returns {void}
+ */
+HelpView.prototype.toggle = function () {
+  this.node.toggle();
+};
+
+/**
+ * Hide the view.
+ *
+ * @returns {void}
+ */
+HelpView.prototype.hide = function () {
+  this.node.hide();
+};
+
+/**
+ * Check to see if the view is visible.
+ *
+ * @returns {Boolean}
+ * Truthy if the view is visible, falsey otherwise.
+ */
+HelpView.prototype.isVisible = function () {
+  return this.node.visible;
+};
+
+
 module.exports = HelpView;