From 7706f62ba018a1e344cc0653e534f472b07ee3d3 Mon Sep 17 00:00:00 2001
From: Marcin Rataj <lidel@lidel.org>
Date: Thu, 14 Jun 2018 13:41:58 +0200
Subject: [PATCH] fix: hide unused options when embedded node is active

---
 add-on/src/options/forms/gateways-form.js  | 67 ++++++++++++----------
 add-on/src/options/forms/ipfs-node-form.js |  3 +-
 add-on/src/options/page.js                 |  1 +
 3 files changed, 39 insertions(+), 32 deletions(-)

diff --git a/add-on/src/options/forms/gateways-form.js b/add-on/src/options/forms/gateways-form.js
index 173b178ff..99575463f 100644
--- a/add-on/src/options/forms/gateways-form.js
+++ b/add-on/src/options/forms/gateways-form.js
@@ -6,6 +6,7 @@ const html = require('choo/html')
 const { normalizeGatewayURL } = require('../../lib/options')
 
 function gatewaysForm ({
+  ipfsNodeType,
   customGatewayUrl,
   useCustomGateway,
   publicGatewayUrl,
@@ -19,37 +20,41 @@ function gatewaysForm ({
     <form>
       <fieldset>
         <legend>${browser.i18n.getMessage('option_header_gateways')}</legend>
-        <div>
-          <label for="customGatewayUrl">
-            <dl>
-              <dt>${browser.i18n.getMessage('option_customGatewayUrl_title')}</dt>
-              <dd>${browser.i18n.getMessage('option_customGatewayUrl_description')}</dd>
-            </dl>
-          </label>
-          <input
-            id="customGatewayUrl"
-            type="url"
-            inputmode="url"
-            required
-            pattern="^https?://[^/]+$"
-            spellcheck="false"
-            title="Enter URL without any sub-path"
-            onchange=${onCustomGatewayUrlChange}
-            value=${customGatewayUrl} />
-        </div>
-        <div>
-          <label for="useCustomGateway">
-            <dl>
-              <dt>${browser.i18n.getMessage('option_useCustomGateway_title')}</dt>
-              <dd>${browser.i18n.getMessage('option_useCustomGateway_description')}</dd>
-            </dl>
-          </label>
-          <input
-            id="useCustomGateway"
-            type="checkbox"
-            onchange=${onUseCustomGatewayChange}
-            checked=${useCustomGateway} />
-        </div>
+          ${ipfsNodeType === 'external' ? html`
+            <div>
+              <label for="customGatewayUrl">
+                <dl>
+                  <dt>${browser.i18n.getMessage('option_customGatewayUrl_title')}</dt>
+                  <dd>${browser.i18n.getMessage('option_customGatewayUrl_description')}</dd>
+                </dl>
+              </label>
+              <input
+                id="customGatewayUrl"
+                type="url"
+                inputmode="url"
+                required
+                pattern="^https?://[^/]+$"
+                spellcheck="false"
+                title="Enter URL without any sub-path"
+                onchange=${onCustomGatewayUrlChange}
+                value=${customGatewayUrl} />
+            </div>
+          ` : null}
+          ${ipfsNodeType === 'external' ? html`
+            <div>
+              <label for="useCustomGateway">
+                <dl>
+                  <dt>${browser.i18n.getMessage('option_useCustomGateway_title')}</dt>
+                  <dd>${browser.i18n.getMessage('option_useCustomGateway_description')}</dd>
+                </dl>
+              </label>
+              <input
+                id="useCustomGateway"
+                type="checkbox"
+                onchange=${onUseCustomGatewayChange}
+                checked=${useCustomGateway} />
+            </div>
+          ` : null}
         <div>
           <label for="publicGatewayUrl">
             <dl>
diff --git a/add-on/src/options/forms/ipfs-node-form.js b/add-on/src/options/forms/ipfs-node-form.js
index a61e187e1..c4fd88aea 100644
--- a/add-on/src/options/forms/ipfs-node-form.js
+++ b/add-on/src/options/forms/ipfs-node-form.js
@@ -17,7 +17,8 @@ function ipfsNodeForm ({ ipfsNodeType, ipfsNodeConfig, onOptionChange }) {
             <dl>
               <dt>${browser.i18n.getMessage('option_ipfsNodeType_title')}</dt>
               <dd>
-                ${browser.i18n.getMessage('option_ipfsNodeType_description')}
+                <p>${browser.i18n.getMessage('option_ipfsNodeType_external_description')}</p>
+                <p>${browser.i18n.getMessage('option_ipfsNodeType_embedded_description')}</p>
                 <p><a href="https://github.com/ipfs-shipyard/ipfs-companion/blob/master/docs/node-types.md" target="_blank">
                   ${browser.i18n.getMessage('option_legend_readMore')}
                 </a></p>
diff --git a/add-on/src/options/page.js b/add-on/src/options/page.js
index b60cb500d..d04c09940 100644
--- a/add-on/src/options/page.js
+++ b/add-on/src/options/page.js
@@ -36,6 +36,7 @@ module.exports = function optionsPage (state, emit) {
         onOptionChange
       })}
       ${gatewaysForm({
+        ipfsNodeType: state.options.ipfsNodeType,
         customGatewayUrl: state.options.customGatewayUrl,
         useCustomGateway: state.options.useCustomGateway,
         publicGatewayUrl: state.options.publicGatewayUrl,