fix: choosing transactions period in Firefox (#137)

Fixes #119 REF #119 (comment)
fmaclen · Oct 27, 2022 · 8c1e68a · 8c1e68a
1 parent 67ae606
commit 8c1e68a
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 9 deletions.
diff --git a/sveltekit/package-lock.json b/sveltekit/package-lock.json
diff --git a/sveltekit/package.json b/sveltekit/package.json
@@ -29,7 +29,7 @@
 		"prettier": "^2.6.2",
 		"prettier-plugin-svelte": "^2.7.0",
 		"sass": "^1.53.0",
-		"svelte": "^3.50.1",
+		"svelte": "^3.52.0",
 		"svelte-check": "^2.9.1",
 		"svelte-preprocess": "^4.10.7",
 		"tslib": "^2.3.1",

diff --git a/sveltekit/src/routes/transactions/+page.svelte b/sveltekit/src/routes/transactions/+page.svelte
@@ -214,7 +214,14 @@
 					options={periods}
 					bind:value={periodIndex}
 					on:change={async () => {
-						await getTransactions();
+						// HACK: there is a race condition (in Firefox) when changing the period.
+						// `periodIndex`, `dateFrom` and `dateTo` are not updated before
+						// getTransactions() is called. Adding a 1ms delay fixes the issue.
+						//
+						// REF https://github.com/Canutin/desktop/issues/119#issuecomment-1293639150
+						setTimeout(async () => {
+							await getTransactions();
+						}, 1);
 					}}
 				/>
 				<div class="transactions__summary">