Do not hide help dialog on click (on options page).

Normally, we close the help dialog when the user clicks outside of it.

On the options page, however, the user might want the help dialog open
while they type command names into the key-mappings input.

Therefore, we disable hide-on-click for this specific use case.

This is a rather unfortunate hack.  However, #2045 is a serious problem
for an important use case.

Fixes #2045.

2 files changed, 10 insertions, 2 deletions

diff --git a/pages/help_dialog.coffee b/pages/help_dialog.coffee
index e16fd5d3..997ea915 100644
--- a/pages/help_dialog.coffee
+++ b/pages/help_dialog.coffee
@@ -27,11 +27,19 @@ HelpDialog =
       HelpDialog.toggleAdvancedCommands, false)
 
     document.documentElement.addEventListener "click", (event) =>
-      @hide() unless @dialogElement.contains event.target
+      # Normally, we hide the help dialog on "click".  On the options page though, we do not.  This allows the
+      # user to view the help page while typing command names into the key mappings input; see #2045.
+      @hide() unless @isVimiumOptionsPage() or @dialogElement.contains event.target
     , false
 
   isReady: -> true
 
+  isVimiumOptionsPage: ->
+    try
+      window.top.isVimiumOptionsPage
+    catch
+      false
+
   show: (html) ->
     for own placeholder, htmlString of html
       @dialogElement.querySelector("#help-dialog-#{placeholder}").innerHTML = htmlString
diff --git a/pages/options.coffee b/pages/options.coffee
index 6070400e..c708efa7 100644
--- a/pages/options.coffee
+++ b/pages/options.coffee
@@ -322,4 +322,4 @@ document.addEventListener "DOMContentLoaded", ->
 
 # Exported for tests.
 root = exports ? window
-root.Options = Options
+extend root, {Options, isVimiumOptionsPage: true}