From 1a0be4130ea1a9c66662c24465be5d97e9891feb Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Wed, 27 May 2026 14:12:01 +0200
Subject: [PATCH] fix(persons): make the show-all switch accessible name match
 its visible text
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The role="switch" toggle set a fixed aria-label of "Zu prüfen (N)" while its
visible text flips to "Alle anzeigen" when active — a visible-text /
accessible-name mismatch (WCAG 2.5.3 Label in Name). Drop the aria-label so
the visible text is the accessible name; aria-checked carries the state.

Refs #667

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
---
 frontend/src/lib/person/PersonFilterBar.svelte | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/frontend/src/lib/person/PersonFilterBar.svelte b/frontend/src/lib/person/PersonFilterBar.svelte
index 686dffe9..ae2bf9c2 100644
--- a/frontend/src/lib/person/PersonFilterBar.svelte
+++ b/frontend/src/lib/person/PersonFilterBar.svelte
@@ -132,11 +132,13 @@ const chipInactive = 'border-line bg-surface text-ink hover:bg-muted';
 
 	<!-- Show-all / Zu prüfen toggle: transcriber-only, reveals the import noise. -->
 	{#if canWrite}
+		<!-- No aria-label: the visible text IS the accessible name (WCAG 2.5.3 Label in Name).
+		     It flips between "Zu prüfen (N)" (off) and "Alle anzeigen" (on) and aria-checked
+		     carries the toggle state, so the announced name always matches what the user reads. -->
 		<button
 			type="button"
 			role="switch"
 			aria-checked={review}
-			aria-label={m.persons_toggle_needs_review({ count: needsReviewCount })}
 			class={[chipBase, 'sm:ml-auto', review ? chipActive : chipInactive]}
 			onclick={() => setReview(!review)}
 		>