Merge pull request #586 from ipfs-shipyard/fix-toggle-setting-copy

fix: make global toggle checkbox less ambiguous
ipfs · Sep 19, 2018 · 83f224c · 83f224c
2 parents d47cc80 + 602769e
commit 83f224c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/add-on/_locales/en/messages.json b/add-on/_locales/en/messages.json
@@ -8,7 +8,7 @@
     "description": "A label for IPFS icon (panel_headerIpfsNodeIconLabel)"
   },
   "panel_headerActiveToggleTitle": {
-    "message": "Global toggle: activate or suspend all IPFS integrations",
+    "message": "Global toggle: Suspend all IPFS integrations",
     "description": "A label for an embedded IPFS node (panel_headerActiveToggleTitle)"
   },
   "panel_statusOffline": {

diff --git a/add-on/src/options/forms/global-toggle-form.js b/add-on/src/options/forms/global-toggle-form.js
@@ -5,11 +5,11 @@ const browser = require('webextension-polyfill')
 const html = require('choo/html')
 
 function globalToggleForm ({ active, onOptionChange }) {
-  const toggle = onOptionChange('active')
+  const toggle = onOptionChange('active', checked => !checked)
   return html`
     <form class="dib mb3">
-      <label for="active" class="dib pa3 pointer ${!active ? 'bg-aqua-muted br2' : ''}">
-        <input class="mr2 pointer" id="active" type="checkbox" onchange=${toggle} checked=${active} />
+      <label for="active" class="dib pa3 pointer ${!active ? 'charcoal bg-gray-muted br2' : ''}">
+        <input class="mr2 pointer" id="active" type="checkbox" onchange=${toggle} checked=${!active} />
         ${browser.i18n.getMessage('panel_headerActiveToggleTitle')}
       </label>
     </form>