From 5ea668d5490226e78958c6e72b348da9f097ac88 Mon Sep 17 00:00:00 2001 From: Adam Malczewski Date: Mon, 1 Jun 2026 09:45:43 +0900 Subject: a11y(sidebar): label the remove-panel button for screen readers MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Gemini review nit. The ✕ button on each sidebar slot (idx > 0) was read by screen readers as "multiplication sign" or "cross". Adds aria-label="Remove panel" so the action is announced clearly. Also gitignore claude-report.md (Gemini review artifact, not source). --- .gitignore | 3 +++ packages/frontend/src/lib/components/SidebarPanel.svelte | 1 + 2 files changed, 4 insertions(+) diff --git a/.gitignore b/.gitignore index 51c3006..960e309 100644 --- a/.gitignore +++ b/.gitignore @@ -23,3 +23,6 @@ packaging/electron/pkg/ packaging/electron/*.pkg.tar.zst packaging/electron/*.tar.zst packages/frontend/release/ + +# Code-review artifacts +claude-report.md diff --git a/packages/frontend/src/lib/components/SidebarPanel.svelte b/packages/frontend/src/lib/components/SidebarPanel.svelte index 66fa6a4..491b1bd 100644 --- a/packages/frontend/src/lib/components/SidebarPanel.svelte +++ b/packages/frontend/src/lib/components/SidebarPanel.svelte @@ -139,6 +139,7 @@ function contentClass(_selected: string): string {