From f7b5df4db830f3cafbf511de00941a8ce4258e20 Mon Sep 17 00:00:00 2001
From: Schalli <kschaller@ctl.de>
Date: Mon, 21 Sep 2026 14:56:02 +0200
Subject: [PATCH] fix(quick-260921-jt4): zwei Schaltflaechenreihen tragen
 role="toolbar" (D-03)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

calculator-widget.tsx Speicherzeile und favorites-widget.tsx
Ansichtsumschalter waren schlichte <div> mit aria-label, das die Rolle
generic stillschweigend verwarf. role="toolbar" ergaenzt (geprueft
sauber; role="group"/"region" loesen useSemanticElements neu aus).

Taschenrechner: fest verdrahteter Text wandert in
calculator.memoryLabel; bei dieser Gelegenheit auch Anzeigefeld
(displayLabel) und Rueckschritt-Taste (backspaceLabel) in den Katalog
gezogen (D-05), da diese Datei ohnehin geaendert wird.

Favoriten: die bisherige Beschriftung ("Favoriten") war sachlich
falsch fuer einen Listen/Kachel-Umschalter — neuer, zutreffender
Schluessel favorites.viewModeLabel statt des wiederverwendeten
favorites.name.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TPPB4ApQxzSU1rwV2Ffj9J
---
 .../dashboard/widgets/calculator-widget.test.tsx   | 14 ++++++++++++--
 .../dashboard/widgets/calculator-widget.tsx        | 10 ++++++----
 .../dashboard/widgets/favorites-widget.tsx         | 11 ++++++++++-
 apps/web/src/messages/de.json                      |  8 ++++++--
 apps/web/src/messages/en.json                      |  8 ++++++--
 5 files changed, 40 insertions(+), 11 deletions(-)

diff --git a/apps/web/src/components/dashboard/widgets/calculator-widget.test.tsx b/apps/web/src/components/dashboard/widgets/calculator-widget.test.tsx
index 6514190..ce53642 100644
--- a/apps/web/src/components/dashboard/widgets/calculator-widget.test.tsx
+++ b/apps/web/src/components/dashboard/widgets/calculator-widget.test.tsx
@@ -1,9 +1,19 @@
 import { render, screen, fireEvent } from '@testing-library/react';
 import { describe, expect, it, vi } from 'vitest';
 
-// Mock next-intl — returns the key so tests are locale-agnostic
+// Mock next-intl — returns the real de.json strings for the few keys this
+// widget actually renders as accessible names/labels, the raw key otherwise
+// (locale-agnostic fallback for anything not explicitly mapped here).
 vi.mock('next-intl', () => ({
-  useTranslations: () => (key: string) => key,
+  useTranslations: () => (key: string) => {
+    const map: Record<string, string> = {
+      'calculator.name': 'Taschenrechner',
+      'calculator.displayLabel': 'Anzeige',
+      'calculator.memoryLabel': 'Speicherfunktionen',
+      'calculator.backspaceLabel': 'Rückschritt',
+    };
+    return map[key] ?? key;
+  },
 }));
 
 // Must import after mocks
diff --git a/apps/web/src/components/dashboard/widgets/calculator-widget.tsx b/apps/web/src/components/dashboard/widgets/calculator-widget.tsx
index 56c61b9..f137571 100644
--- a/apps/web/src/components/dashboard/widgets/calculator-widget.tsx
+++ b/apps/web/src/components/dashboard/widgets/calculator-widget.tsx
@@ -336,13 +336,15 @@ export function CalculatorWidget({ isEditMode }: WidgetProps) {
       {/* Display */}
       <output
         className="flex min-h-[2.5rem] items-end justify-end rounded bg-muted px-3 py-1 font-mono text-[clamp(14px,min(8cqw,10cqh),96px)] font-semibold text-foreground overflow-hidden"
-        aria-label="Anzeige"
+        aria-label={t('calculator.displayLabel')}
       >
         <span className="truncate">{display}</span>
       </output>
 
-      {/* Memory row */}
-      <div className="flex gap-1" aria-label="Speicherfunktionen">
+      {/* Memory row — role="toolbar" (D-03): ohne Rolle ist das <div> generic
+          und die Vorlesehilfe verwirft aria-label stillschweigend. Geprueft
+          sauber (role="group"/"region" loesen useSemanticElements neu aus). */}
+      <div className="flex gap-1" role="toolbar" aria-label={t('calculator.memoryLabel')}>
         <button type="button" className={memBtn} onClick={memoryClear} disabled={memory === 0}>MC</button>
         <button type="button" className={memBtn} onClick={memoryRecall} disabled={memory === 0}>MR</button>
         <button type="button" className={memBtn} onClick={memoryAdd}>M+</button>
@@ -356,7 +358,7 @@ export function CalculatorWidget({ isEditMode }: WidgetProps) {
         <CalcButton label="%" className={utilBtn} onClick={() => applyUnary('percent')} />
         <CalcButton label="CE" className={utilBtn} onClick={clearEntry} />
         <CalcButton label="C" className={utilBtn} onClick={clearAll} />
-        <CalcButton label="⌫" ariaLabel="Rückschritt" className={utilBtn} onClick={backspace} />
+        <CalcButton label="⌫" ariaLabel={t('calculator.backspaceLabel')} className={utilBtn} onClick={backspace} />
 
         {/* Row 2: unary + divide */}
         <CalcButton label="1/x" className={utilBtn} onClick={() => applyUnary('reciprocal')} />
diff --git a/apps/web/src/components/dashboard/widgets/favorites-widget.tsx b/apps/web/src/components/dashboard/widgets/favorites-widget.tsx
index fb8220f..804bdf8 100644
--- a/apps/web/src/components/dashboard/widgets/favorites-widget.tsx
+++ b/apps/web/src/components/dashboard/widgets/favorites-widget.tsx
@@ -263,7 +263,16 @@ export function FavoritesWidget({
       <div className="flex flex-1 flex-col gap-2 overflow-auto p-1">
       {/* View mode toggle (edit mode only) */}
       {isEditMode && (
-        <div className="flex gap-1 widgetNoDrag" aria-label={t('favorites.name')}>
+        // role="toolbar" (D-03): die bisherige Beschriftung ("Favoriten")
+        // war sachlich falsch fuer einen Umschalter zwischen Listen- und
+        // Kachelansicht — hier greift D-03 im Sinn des Streichens (die
+        // Beschriftung trug keine echte Bedeutung), eine richtige
+        // Beschriftung ist trotzdem besser als gar keine.
+        <div
+          className="flex gap-1 widgetNoDrag"
+          role="toolbar"
+          aria-label={t('favorites.viewModeLabel')}
+        >
           <button
             type="button"
             onClick={() => handleViewMode('list')}
diff --git a/apps/web/src/messages/de.json b/apps/web/src/messages/de.json
index a82028a..e6beb73 100644
--- a/apps/web/src/messages/de.json
+++ b/apps/web/src/messages/de.json
@@ -300,7 +300,10 @@
     },
     "calculator": {
       "name": "Taschenrechner",
-      "description": "Grundrechenarten mit Tastatureingabe"
+      "description": "Grundrechenarten mit Tastatureingabe",
+      "displayLabel": "Anzeige",
+      "memoryLabel": "Speicherfunktionen",
+      "backspaceLabel": "Rückschritt"
     },
     "favorites": {
       "name": "Favoriten",
@@ -320,7 +323,8 @@
       "cancelButton": "Abbrechen",
       "error": "Fehler beim Laden der Favoriten",
       "titleLabel": "Titel",
-      "titlePlaceholder": "Titel (optional)"
+      "titlePlaceholder": "Titel (optional)",
+      "viewModeLabel": "Ansicht wechseln"
     },
     "stopwatch": {
       "name": "Stoppuhr",
diff --git a/apps/web/src/messages/en.json b/apps/web/src/messages/en.json
index 6ab4ca7..47472f5 100644
--- a/apps/web/src/messages/en.json
+++ b/apps/web/src/messages/en.json
@@ -300,7 +300,10 @@
     },
     "calculator": {
       "name": "Calculator",
-      "description": "Basic arithmetic with keyboard support"
+      "description": "Basic arithmetic with keyboard support",
+      "displayLabel": "Display",
+      "memoryLabel": "Memory functions",
+      "backspaceLabel": "Backspace"
     },
     "favorites": {
       "name": "Favorites",
@@ -320,7 +323,8 @@
       "cancelButton": "Cancel",
       "error": "Error loading favorites",
       "titleLabel": "Title",
-      "titlePlaceholder": "Title (optional)"
+      "titlePlaceholder": "Title (optional)",
+      "viewModeLabel": "Switch view"
     },
     "stopwatch": {
       "name": "Stopwatch",