Avoid double loader

fyreplace · Oct 18, 2024 · 9e21733 · 9e21733
1 parent b1e92f3
commit 9e21733
Showing 1 changed file with 5 additions and 11 deletions.
diff --git a/src/routes/settings/+page.svelte b/src/routes/settings/+page.svelte
@@ -65,16 +65,11 @@
 	function removeAvatar() {
 		return call(
 			async () => {
-				try {
-					isLoadingAvatar = true;
-					const client = await getUsersClient();
-					await client.deleteCurrentUserAvatar();
+				const client = await getUsersClient();
+				await client.deleteCurrentUserAvatar();
 
-					if (currentUser) {
-						currentUser = { ...currentUser, avatar: '' };
-					}
-				} finally {
-					isLoadingAvatar = false;
+				if (currentUser) {
+					currentUser = { ...currentUser, avatar: '' };
 				}
 			},
 			async () => new DisplayableError()
@@ -108,8 +103,7 @@
 					<td>
 						<Button
 							type="button"
-							disabled={!currentUser?.avatar}
-							loading={isLoadingAvatar}
+							disabled={!currentUser?.avatar || isLoadingAvatar}
 							on:click={removeAvatar}
 						>
 							{t('settings.profile.avatar.remove')}