From 8cf72842ad516cba7eb161ae650d48a298e55442 Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Sat, 20 Feb 2016 07:01:54 +0000
Subject: hideHUD option applies to insert mode only.

This makes the `hideHud` option apply only to insert mode (when entered with `i`).

Fixes #1953.
Fixes #487.

We could rename the option itself and add migration code, but that seems overkill.

An alternative would be to remove this option entirely.
---
 content_scripts/hud.coffee         | 3 +--
 content_scripts/mode_insert.coffee | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

(limited to 'content_scripts')

diff --git a/content_scripts/hud.coffee b/content_scripts/hud.coffee
index 5a3d9b79..453d6522 100644
--- a/content_scripts/hud.coffee
+++ b/content_scripts/hud.coffee
@@ -87,8 +87,7 @@ HUD =
     DomUtils.documentReady -> ready = true
     -> ready and document.body != null
 
-  # A preference which can be toggled in the Options page. */
-  enabled: -> !Settings.get("hideHud")
+  enabled: -> true
 
 class Tween
   opacity: 0
diff --git a/content_scripts/mode_insert.coffee b/content_scripts/mode_insert.coffee
index a35633ac..93615b8f 100644
--- a/content_scripts/mode_insert.coffee
+++ b/content_scripts/mode_insert.coffee
@@ -30,7 +30,7 @@ class InsertMode extends Mode
 
     defaults =
       name: "insert"
-      indicator: if @permanent then null else "Insert mode"
+      indicator: if not @permanent and not Settings.get "hideHud"  then "Insert mode"
       keypress: handleKeyEvent
       keyup: handleKeyEvent
       keydown: handleKeyEvent
-- 
cgit v1.2.3