fix: ensure async function finish before window closes

quoid · Jan 30, 2024 · b514e9b · b514e9b
1 parent 08ab4f7
commit b514e9b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/src/ext/action-popup/App.svelte b/src/ext/action-popup/App.svelte
@@ -484,6 +484,11 @@
 		}
 	});
 
+	async function gotoExtensionPage() {
+		await openExtensionPage();
+		window.close();
+	}
+
 	/**
 	 * Temporary settings page entrance for beta test (iOS)
 	 * @todo new permanent button will be added via popup refactoring
@@ -524,7 +529,7 @@
 {/if}
 {#if showBetaNews && platform !== "macos"}
 	<div class="warn">
-		NEW: <button on:click={openExtensionPage}><b>Settings page</b></button> is
+		NEW: <button on:click={gotoExtensionPage}><b>Settings page</b></button> is
 		now available on iOS!
 		<IconButton
 			icon={iconClear}
@@ -594,13 +599,7 @@
 </div>
 {#if !inactive && platform === "macos"}
 	<div class="footer">
-		<button
-			class="link"
-			on:click={() => {
-				openExtensionPage();
-				window.close();
-			}}
-		>
+		<button class="link" on:click={gotoExtensionPage}>
 			Open Extension Page
 		</button>
 	</div>

diff --git a/src/ext/shared/utils.js b/src/ext/shared/utils.js
@@ -373,8 +373,8 @@ export async function openExtensionPage() {
 	const tabs = await browser.tabs.query({ url });
 	const tab = tabs.find((e) => e.url.startsWith(url));
 	if (!tab) return browser.tabs.create({ url });
-	browser.tabs.update(tab.id, { active: true });
-	browser.windows.update(tab.windowId, { focused: true });
+	await browser.tabs.update(tab.id, { active: true });
+	await browser.windows.update(tab.windowId, { focused: true });
 }
 
 /**