[taler-wallet-core] 06/06: fix: shorcut for wallet #6592

[gnunet]

This is an automated email from the git hooks/post-receive script.

sebasjm pushed a commit to branch master
in repository wallet-core.

commit 9b85d139bf7bdc360ea0894e09f6115cd9d472d8
Author: Sebastian <sebasjm@gmail.com>
AuthorDate: Tue Apr 26 11:48:23 2022 -0300

    fix: shorcut for wallet #6592
---
 .../taler-wallet-webextension/manifest-v2.json     |  7 +++++-
 .../taler-wallet-webextension/manifest-v3.json     |  7 +++++-
 .../src/platform/chrome.ts                         |  2 +-
 packages/taler-wallet-webextension/src/stories.tsx | 15 ++++++++++++
 .../src/wallet/Welcome.tsx                         | 28 +++++++++++++++++++++-
 5 files changed, 55 insertions(+), 4 deletions(-)

diff --git a/packages/taler-wallet-webextension/manifest-v2.json 
b/packages/taler-wallet-webextension/manifest-v2.json
index e92217b9..f98f70e6 100644
--- a/packages/taler-wallet-webextension/manifest-v2.json
+++ b/packages/taler-wallet-webextension/manifest-v2.json
@@ -17,6 +17,11 @@
     "32": "static/img/icon.png",
     "128": "static/img/logo.png"
   },
+  "commands": {
+    "_execute_browser_action": {
+      "suggested_key": "Alt+W"
+    }
+  },
   "permissions": [
     "unlimitedStorage",
     "activeTab"
@@ -30,7 +35,7 @@
     "default_icon": {
       "32": "static/img/icon.png"
     },
-    "default_title": "Taler",
+    "default_title": "GNU Taler Wallet",
     "default_popup": "static/popup.html"
   },
   "background": {
diff --git a/packages/taler-wallet-webextension/manifest-v3.json 
b/packages/taler-wallet-webextension/manifest-v3.json
index 576b263f..a8f043b3 100644
--- a/packages/taler-wallet-webextension/manifest-v3.json
+++ b/packages/taler-wallet-webextension/manifest-v3.json
@@ -16,6 +16,11 @@
     "scripting",
     "alarms"
   ],
+  "commands": {
+    "_execute_action": {
+      "suggested_key": "Alt+W"
+    }
+  },
   "optional_permissions": [
     "webRequest"
   ],
@@ -27,7 +32,7 @@
     "default_icon": {
       "32": "static/img/icon.png"
     },
-    "default_title": "Taler",
+    "default_title": "GNU Taler Wallet",
     "default_popup": "static/popup.html"
   },
   "background": {
diff --git a/packages/taler-wallet-webextension/src/platform/chrome.ts 
b/packages/taler-wallet-webextension/src/platform/chrome.ts
index 67779f08..d1f4585d 100644
--- a/packages/taler-wallet-webextension/src/platform/chrome.ts
+++ b/packages/taler-wallet-webextension/src/platform/chrome.ts
@@ -304,7 +304,7 @@ function registerOnInstalled(callback: () => void): void {
   // This needs to be outside of main, as Firefox won't fire the event if
   // the listener isn't created synchronously on loading the backend.
   chrome.runtime.onInstalled.addListener((details) => {
-    console.log("onInstalled with reason", details.reason);
+    console.log(`onInstalled with reason: "${details.reason}"`);
     if (details.reason === chrome.runtime.OnInstalledReason.INSTALL) {
       callback()
     }
diff --git a/packages/taler-wallet-webextension/src/stories.tsx 
b/packages/taler-wallet-webextension/src/stories.tsx
index 1ad91a13..9c0f69ec 100644
--- a/packages/taler-wallet-webextension/src/stories.tsx
+++ b/packages/taler-wallet-webextension/src/stories.tsx
@@ -38,6 +38,9 @@ import * as wallet from "./wallet/index.stories.js";
 import * as cta from "./cta/index.stories.js";
 import * as components from "./components/index.stories.js";
 import { strings } from "./i18n/strings.js";
+import { setupPlatform } from "./platform/api.js";
+import chromeAPI from "./platform/chrome.js";
+import firefoxAPI from "./platform/firefox.js";
 
 const url = new URL(window.location.href);
 const lang = url.searchParams.get("lang") || "en";
@@ -441,3 +444,15 @@ function setupLiveReload(port: number, onReload: () => 
void): void {
     console.error(error);
   };
 }
+
+const isFirefox = typeof (window as any)["InstallTrigger"] !== "undefined";
+
+//FIXME: create different entry point for any platform instead of
+//switching in runtime
+if (isFirefox) {
+  console.log("Wallet setup for Firefox API");
+  setupPlatform(firefoxAPI);
+} else {
+  console.log("Wallet setup for Chrome API");
+  setupPlatform(chromeAPI);
+}
diff --git a/packages/taler-wallet-webextension/src/wallet/Welcome.tsx 
b/packages/taler-wallet-webextension/src/wallet/Welcome.tsx
index 5aad3b78..58023a20 100644
--- a/packages/taler-wallet-webextension/src/wallet/Welcome.tsx
+++ b/packages/taler-wallet-webextension/src/wallet/Welcome.tsx
@@ -27,6 +27,7 @@ import { SubTitle, Title } from 
"../components/styled/index.js";
 import { useTranslationContext } from "../context/translation.js";
 import { useDiagnostics } from "../hooks/useDiagnostics.js";
 import { useExtendedPermissions } from "../hooks/useExtendedPermissions.js";
+import { platform } from "../platform/api.js";
 
 export function WelcomePage(): VNode {
   const [permissionsEnabled, togglePermissions] = useExtendedPermissions();
@@ -61,8 +62,33 @@ export function View({
       </Title>
       <div>
         <p>
-          <i18n.Translate>Thank you for installing the wallet.</i18n.Translate>
+          <i18n.Translate>
+            You can open the GNU Taler Wallet using the combination{" "}
+            <pre style="font-weight: bold; display: 
inline;">&lt;ALT+W&gt;</pre>
+            .
+          </i18n.Translate>
         </p>
+        {!platform.isFirefox() && (
+          <Fragment>
+            <p>
+              <i18n.Translate>
+                Also pinning the GNU Taler Wallet to your Chrome browser allows
+                you to quick access without keyboard:
+              </i18n.Translate>
+            </p>
+            <ol style={{ paddingLeft: 40 }}>
+              <li>
+                <i18n.Translate>Click the puzzle icon</i18n.Translate>
+              </li>
+              <li>
+                <i18n.Translate>Search for GNU Taler Wallet</i18n.Translate>
+              </li>
+              <li>
+                <i18n.Translate>Click the pin icon</i18n.Translate>
+              </li>
+            </ol>
+          </Fragment>
+        )}
         <SubTitle>
           <i18n.Translate>Permissions</i18n.Translate>
         </SubTitle>

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.