Merge pull request #2259 from ethereum/yann300-patch-64

Use addTooltip instead of alert

Merge pull request #2259 from ethereum/yann300-patch-64
67c15b2c · yann300 · GitHub · 2b48c31c · 5b67b80f · 67c15b2c
Unverified Commit 67c15b2c authored Aug 13, 2019 by yann300 Committed by GitHub Aug 13, 2019
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 4 deletions

settings.js src/app/tabs/runTab/settings.js +2 -4

No files found.
--- a/src/app/tabs/runTab/settings.js
+++ b/src/app/tabs/runTab/settings.js
@@ -182,15 +182,13 @@ class SettingsUI {
          modal.hide()
          modalDialogCustom.prompt('External node request', 'Web3 Provider Endpoint', 'http://localhost:8545', (target) => {
            this.settings.setProviderFromEndpoint(target, context, (alertMsg) => {
-              if (alertMsg) {
-                modalDialogCustom.alert(alertMsg)
-              }
+              if (alertMsg) addTooltip(alertMsg)
              this.setFinalContext()
            })
          }, this.setFinalContext.bind(this))
        }, this.setFinalContext.bind(this))
      }, (alertMsg) => {
-        modalDialogCustom.alert(alertMsg)
+        addTooltip(alertMsg)
      }, this.setFinalContext.bind(this))
    })