From 004d135d931fd6ed3f843ab502d8e2e9e1fa18ef Mon Sep 17 00:00:00 2001
From: orangemug <info@orangemug.co.uk>
Date: Mon, 24 Sep 2018 14:40:17 +0100
Subject: [PATCH] Show error if style fails to load and disabled button if
 input is empty

---
 src/components/Button.jsx           |  1 +
 src/components/modals/OpenModal.jsx | 28 +++++++++++++++++++++++++---
 src/styles/_components.scss         |  6 ++++++
 3 files changed, 32 insertions(+), 3 deletions(-)

diff --git a/src/components/Button.jsx b/src/components/Button.jsx
index e7690074..294c47fd 100644
--- a/src/components/Button.jsx
+++ b/src/components/Button.jsx
@@ -15,6 +15,7 @@ class Button extends React.Component {
   render() {
     return <button
       onClick={this.props.onClick}
+      disabled={this.props.disabled}
       aria-label={this.props["aria-label"]}
       className={classnames("maputnik-button", this.props.className)}
       data-wd-key={this.props["data-wd-key"]}
diff --git a/src/components/modals/OpenModal.jsx b/src/components/modals/OpenModal.jsx
index 85794e3d..31433db6 100644
--- a/src/components/modals/OpenModal.jsx
+++ b/src/components/modals/OpenModal.jsx
@@ -51,7 +51,9 @@ class OpenModal extends React.Component {
 
   constructor(props) {
     super(props);
-    this.state = {};
+    this.state = {
+      isUrlEmpty: true
+    };
   }
 
   clearError() {
@@ -102,6 +104,7 @@ class OpenModal extends React.Component {
     })
     .catch((err) => {
       this.setState({
+        error: `Failed to load: '${styleUrl}'`,
         activeRequest: null,
         activeRequestUrl: null
       });
@@ -154,6 +157,13 @@ class OpenModal extends React.Component {
     this.props.onOpenToggle();
   }
 
+  onChangeUrl = () => {
+    const length = this.styleUrlElement.value.length;
+    this.setState({
+      isUrlEmpty: length < 1
+    });
+  }
+
   render() {
     const styleOptions = publicStyles.map(style => {
       return <PublicStyle
@@ -197,9 +207,21 @@ class OpenModal extends React.Component {
             <p>
               Load from a URL. Note that the URL must have <a href="https://enable-cors.org" target="_blank" rel="noopener noreferrer">CORS enabled</a>.
             </p>
-            <input data-wd-key="open-modal.url.input" type="text" ref={(input) => this.styleUrlElement = input} className="maputnik-input" placeholder="Enter URL..."/>
+            <input
+              data-wd-key="open-modal.url.input"
+              type="text"
+              ref={(input) => this.styleUrlElement = input}
+              className="maputnik-input"
+              placeholder="Enter URL..."
+              onChange={this.onChangeUrl}
+            />
             <div>
-              <Button data-wd-key="open-modal.url.button" className="maputnik-big-button" onClick={this.onOpenUrl}>Open URL</Button>
+              <Button
+                data-wd-key="open-modal.url.button"
+                className="maputnik-big-button"
+                onClick={this.onOpenUrl}
+                disabled={this.state.isUrlEmpty}
+              >Open URL</Button>
             </div>
           </section>
 
diff --git a/src/styles/_components.scss b/src/styles/_components.scss
index 302ec528..58ab88f0 100644
--- a/src/styles/_components.scss
+++ b/src/styles/_components.scss
@@ -77,6 +77,12 @@
     background-color: lighten($color-midgray, 12);
     color: $color-white;
   }
+
+  &:disabled {
+    background-color: darken($color-midgray, 5);
+    color: $color-midgray;
+    cursor: not-allowed;
+  }
 }
 
 .maputnik-big-button {