From 250fa87578ee1df907b8e449ab1eba745fd6f0c3 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Mon, 9 Feb 2026 21:10:51 +0000
Subject: [PATCH] feat(ui): improve LoadingButton UX and accessibility

- Reset error state on click
- Auto-reset success state after 3s
- Add aria-label and live region for screen readers
- Apply formatting fixes to pass lint

Co-authored-by: Yeboster <23556525+Yeboster@users.noreply.github.com>
---
 .github/dependabot.yml                        |  6 ++--
 .jules/palette.md                             |  3 ++
 src/components/DomainPayment.svelte           |  8 +++--
 src/components/LoadingButton.svelte           | 36 +++++++++++++++++--
 src/components/Record.svelte                  | 14 ++++++--
 src/lib/sdk.ts                                |  2 +-
 src/lib/server/index.ts                       |  1 -
 src/lib/types.ts                              |  8 ++---
 src/lib/url.ts                                |  9 +++--
 src/routes/+layout.svelte                     |  1 -
 .../register/[name]/fees/[coin]/+server.ts    | 19 +++++-----
 src/routes/profile/+page.svelte               |  1 -
 src/styles/mixins.scss                        |  2 +-
 src/theme/dark/_smui-theme.scss               |  2 +-
 14 files changed, 78 insertions(+), 34 deletions(-)
 create mode 100644 .jules/palette.md

diff --git a/.github/dependabot.yml b/.github/dependabot.yml
index 42adb44..301e978 100644
--- a/.github/dependabot.yml
+++ b/.github/dependabot.yml
@@ -5,7 +5,7 @@
 
 version: 2
 updates:
-  - package-ecosystem: "npm"
-    directory: "/"
+  - package-ecosystem: 'npm'
+    directory: '/'
     schedule:
-      interval: "weekly"
+      interval: 'weekly'
diff --git a/.jules/palette.md b/.jules/palette.md
new file mode 100644
index 0000000..87b8e92
--- /dev/null
+++ b/.jules/palette.md
@@ -0,0 +1,3 @@
+## 2024-10-24 - Async Button State Accessibility
+**Learning:** Async buttons with visual-only state changes (spinner, icons) are inaccessible to screen readers without explicit `aria-live` announcements. Success states that persist indefinitely can be confusing and block retry actions.
+**Action:** Always include a visually hidden `span` with `role="status"` and `aria-live="polite"` to announce state changes ("Loading", "Success", "Error"). Implement auto-reset for success states on buttons.
diff --git a/src/components/DomainPayment.svelte b/src/components/DomainPayment.svelte
index ac131ee..e04970d 100644
--- a/src/components/DomainPayment.svelte
+++ b/src/components/DomainPayment.svelte
@@ -102,7 +102,11 @@
 			<h4>{domainName}</h4>
 
 			<div class="years">
-				<IconButton on:click={() => addYears(-1)} disabled={years === 1 || feesApproved} aria-label="remove-year">
+				<IconButton
+					on:click={() => addYears(-1)}
+					disabled={years === 1 || feesApproved}
+					aria-label="remove-year"
+				>
 					<Icon icon="remove" />
 				</IconButton>
 				<span>{years} {yearsLabel}</span>
@@ -111,7 +115,7 @@
 				</IconButton>
 			</div>
 
-	 		<div class="coin">
+			<div class="coin">
 				<p class="title text-center">Payment token</p>
 				<div class="row centered">
 					<Select bind:value={$selectedCoin} label="Select Token" variant="outlined">
diff --git a/src/components/LoadingButton.svelte b/src/components/LoadingButton.svelte
index a96b95b..46a5a1e 100644
--- a/src/components/LoadingButton.svelte
+++ b/src/components/LoadingButton.svelte
@@ -4,10 +4,12 @@
 	import Button, { Label } from '@smui/button';
 	import Icon from 'src/components/Icon.svelte';
 	import CircularProgress from '@smui/circular-progress';
+	import { onDestroy } from 'svelte';
 
 	let className = '';
 	export { className as class };
 	export let onClick: () => Promise<void>;
+	// eslint-disable-next-line @typescript-eslint/no-explicit-any
 	export let onError: (error: any) => Promise<void> = async (error) => {
 		let message;
 		if (error && error instanceof Error) message = error.message;
@@ -24,14 +26,25 @@
 
 	let loading: boolean | undefined;
 	let hasError = false;
+	let resetTimeout: ReturnType<typeof setTimeout>;
+
+	onDestroy(() => {
+		clearTimeout(resetTimeout);
+	});
 
 	async function handleClick() {
 		if (loading) return;
 
+		clearTimeout(resetTimeout);
+
+		hasError = false;
 		loading = true;
 
 		try {
 			await onClick();
+			resetTimeout = setTimeout(() => {
+				loading = undefined;
+			}, 3000);
 		} catch (error) {
 			hasError = true;
 			await onError(error);
@@ -39,18 +52,23 @@
 
 		loading = false;
 	}
+
+	$: statusText = loading ? 'Loading' : hasError ? 'Error' : loading === false ? 'Success' : '';
 </script>
 
 <Button class={className} disabled={isDisabled} on:click={handleClick} {variant}>
-	<Label><slot /></Label>
+	<Label>
+		<slot />
+		<span class="sr-only" role="status" aria-live="polite">{statusText}</span>
+	</Label>
 	{#if loading}
-		<div class="loading">
+		<div class="loading" role="progressbar" aria-label="Loading">
 			<CircularProgress style="height: 20px; width: 20px;" indeterminate />
 		</div>
 	{:else if hasError}
 		<Icon icon="error" align="right" />
 	{:else if loading === false}
-		<Icon icon="done" align="right"/>
+		<Icon icon="done" align="right" />
 	{/if}
 </Button>
 
@@ -62,4 +80,16 @@
 
 		margin-left: 1rem;
 	}
+
+	.sr-only {
+		position: absolute;
+		width: 1px;
+		height: 1px;
+		padding: 0;
+		margin: -1px;
+		overflow: hidden;
+		clip: rect(0, 0, 0, 0);
+		white-space: nowrap;
+		border: 0;
+	}
 </style>
diff --git a/src/components/Record.svelte b/src/components/Record.svelte
index 828f6f2..913b790 100644
--- a/src/components/Record.svelte
+++ b/src/components/Record.svelte
@@ -28,7 +28,7 @@
 	$: errors = invalid ? validator.getErrors() : [];
 	$: disabled = !edit;
 	$: validator = getValidator(klass);
-	$: maxLength = 'maxLength' in validator.rules ? validator.rules['maxLength'] as number : 64
+	$: maxLength = 'maxLength' in validator.rules ? (validator.rules['maxLength'] as number) : 64;
 
 	let edit = false;
 
@@ -99,10 +99,18 @@
 		</div>
 	{:else if editMode}
 		<div class="actions">
-			<IconButton on:click={() => toggleEdit()} disabled={!$walletConnected} aria-label="edit-record">
+			<IconButton
+				on:click={() => toggleEdit()}
+				disabled={!$walletConnected}
+				aria-label="edit-record"
+			>
 				<Icon icon="edit" />
 			</IconButton>
-			<IconButton on:click={() => (dialogOpen = true)} disabled={!$walletConnected} aria-label="delete-record">
+			<IconButton
+				on:click={() => (dialogOpen = true)}
+				disabled={!$walletConnected}
+				aria-label="delete-record"
+			>
 				<Icon icon="delete" />
 			</IconButton>
 		</div>
diff --git a/src/lib/sdk.ts b/src/lib/sdk.ts
index 5480675..3690a64 100644
--- a/src/lib/sdk.ts
+++ b/src/lib/sdk.ts
@@ -22,4 +22,4 @@ export const profileRecords = [
 	RecordClassEnum.Price
 ].map((v) => RecordClassEnum[v]);
 
-export const getValidator = (klass: string) => getRecordValidator(getRecordClassFrom(klass))
+export const getValidator = (klass: string) => getRecordValidator(getRecordClassFrom(klass));
diff --git a/src/lib/server/index.ts b/src/lib/server/index.ts
index d776b94..d87bace 100644
--- a/src/lib/server/index.ts
+++ b/src/lib/server/index.ts
@@ -57,5 +57,4 @@ export const getStats = async (): Promise<DomainStats> => {
 	};
 };
 
-
 export const apiError = (message: string, status = 400) => json({ error: message }, { status });
diff --git a/src/lib/types.ts b/src/lib/types.ts
index 1d202e1..b57ab3e 100644
--- a/src/lib/types.ts
+++ b/src/lib/types.ts
@@ -42,8 +42,8 @@ export interface DomainPaymentParams {
 }
 
 export interface DomainFeesResponse {
-	feesLabel: number
-	fees: string
-	symbol: string
-	address: string
+	feesLabel: number;
+	fees: string;
+	symbol: string;
+	address: string;
 }
diff --git a/src/lib/url.ts b/src/lib/url.ts
index ede74eb..710d0a9 100644
--- a/src/lib/url.ts
+++ b/src/lib/url.ts
@@ -8,12 +8,11 @@ export const explorerAddressUrl = (address: string) => {
 	if (address.startsWith('00'))
 		// Account
 		url += `/accounts/${address}/assets`;
-	else
-		// Contract
-		url += `/contracts/${address}`;
+	// Contract
+	else url += `/contracts/${address}`;
 
-	return url
-}
+	return url;
+};
 
 export const bridgeUrl = `${config.browserUrl}/bridge`;
 
diff --git a/src/routes/+layout.svelte b/src/routes/+layout.svelte
index e491643..a5e8638 100644
--- a/src/routes/+layout.svelte
+++ b/src/routes/+layout.svelte
@@ -201,5 +201,4 @@
 		align-items: center;
 		height: 100%;
 	}
-
 </style>
diff --git a/src/routes/api/register/[name]/fees/[coin]/+server.ts b/src/routes/api/register/[name]/fees/[coin]/+server.ts
index 72763fd..af646a0 100644
--- a/src/routes/api/register/[name]/fees/[coin]/+server.ts
+++ b/src/routes/api/register/[name]/fees/[coin]/+server.ts
@@ -4,14 +4,17 @@ import { json } from '@sveltejs/kit';
 import type { DomainFeesResponse } from 'src/lib/types';
 
 export async function GET({ params: { name, coin } }) {
-  return handleError(async () => {
-    const validCoins = metaNamesSdk.config.byoc.map(byoc => byoc.symbol.toString())
-    if (!(validCoins.includes(coin))) return apiError('Invalid coin');
+	return handleError(async () => {
+		const validCoins = metaNamesSdk.config.byoc.map((byoc) => byoc.symbol.toString());
+		if (!validCoins.includes(coin)) return apiError('Invalid coin');
 
-    const normalizedDomain = metaNamesSdk.domainRepository.domainValidator.normalize(name)
-    const domainFees = await metaNamesSdk.domainRepository.calculateMintFees(normalizedDomain, coin as BYOCSymbol);
-    const fees = { ...domainFees, fees: domainFees.fees.toString() }
+		const normalizedDomain = metaNamesSdk.domainRepository.domainValidator.normalize(name);
+		const domainFees = await metaNamesSdk.domainRepository.calculateMintFees(
+			normalizedDomain,
+			coin as BYOCSymbol
+		);
+		const fees = { ...domainFees, fees: domainFees.fees.toString() };
 
-    return json(fees);
-  });
+		return json(fees);
+	});
 }
diff --git a/src/routes/profile/+page.svelte b/src/routes/profile/+page.svelte
index 8b9fd1d..7f29073 100644
--- a/src/routes/profile/+page.svelte
+++ b/src/routes/profile/+page.svelte
@@ -74,7 +74,6 @@
 </div>
 
 <style lang="scss">
-
 	.domains {
 		margin-top: 1.5rem;
 		margin-bottom: 0;
diff --git a/src/styles/mixins.scss b/src/styles/mixins.scss
index c1497d9..7d94a8f 100644
--- a/src/styles/mixins.scss
+++ b/src/styles/mixins.scss
@@ -1,4 +1,4 @@
-@use "sass:string";
+@use 'sass:string';
 
 $amounts: (0, 1, 2, 3, 4);
 $types: (top, bottom, left, right);
diff --git a/src/theme/dark/_smui-theme.scss b/src/theme/dark/_smui-theme.scss
index 3331f5b..48d0627 100644
--- a/src/theme/dark/_smui-theme.scss
+++ b/src/theme/dark/_smui-theme.scss
@@ -5,7 +5,7 @@ $text-color: #d5ccff;
 @use '@material/theme/color-palette';
 @use '@material/theme/index' as theme with (
 	$primary: #6849fe,
-	$secondary: #D0C7FF,
+	$secondary: #d0c7ff,
 	$surface: color.adjust(color-palette.$grey-900, $blue: +4),
 	$background: #363535,
 	$error: color-palette.$red-700