Update ledger message/action if transport creation was blocked by exi…

…sting connection
MetaMask · Nov 2, 2021 · 4a0d6bf · 4a0d6bf
1 parent 4d04e74
commit 4a0d6bf
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/app/_locales/en/messages.json b/app/_locales/en/messages.json
@@ -1258,8 +1258,8 @@
   "ledgerAccountRestriction": {
     "message": "You need to make use your last account before you can add a new one."
   },
-  "ledgerConnectionInstructionCloseLedgerBridge": {
-    "message": "Make sure other software connected to your device are closed."
+  "ledgerConnectionInstructionCloseOtherApps": {
+    "message": "Close any other software connected to your device and then click here to refresh."
   },
   "ledgerConnectionInstructionHeader": {
     "message": "Prior to clicking confirm:"

diff --git a/ui/components/app/ledger-instruction-field/ledger-instruction-field.js b/ui/components/app/ledger-instruction-field/ledger-instruction-field.js
@@ -147,7 +147,20 @@ export default function LedgerInstructionField({ showDataInstruction }) {
               showDataInstruction,
             )}
             {renderInstructionStep(
-              `- ${t('ledgerConnectionInstructionCloseLedgerBridge')}`,
+              <span>
+                <Button
+                  type="link"
+                  onClick={async () => {
+                    if (environmentTypeIsFullScreen) {
+                      window.location.reload();
+                    } else {
+                      global.platform.openExtensionInBrowser(null, null, true);
+                    }
+                  }}
+                >
+                  {t('ledgerConnectionInstructionCloseOtherApps')}
+                </Button>
+              </span>,
               transportStatus === TRANSPORT_STATES.DEVICE_OPEN_FAILURE,
             )}
             {renderInstructionStep(