From 1e7d8ffe7425c806a6b735ac636aafa86cc76c7f Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Sun, 2 Aug 2026 14:21:27 +0000
Subject: [PATCH 1/4] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20[UX=20improvement?=
 =?UTF-8?q?]=20=EC=A0=91=EA=B7=BC=EC=84=B1=20=EC=9E=88=EB=8A=94=20?=
 =?UTF-8?q?=ED=88=B4=ED=8C=81=EC=9D=84=20=EC=9C=84=ED=95=9C=20disabled=20?=
 =?UTF-8?q?=EC=86=8D=EC=84=B1=20=EA=B0=9C=EC=84=A0?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

💡 What: ScoreView 컴포넌트 내의 "Open score" 버튼에서 native disabled 속성을 제거하고, aria-disabled와 title 툴팁으로 대체했습니다.
🎯 Why: native disabled 속성은 스크린 리더에서 요소를 완전히 숨기고 마우스 이벤트를 차단하여, 비활성화된 이유를 설명하는 툴팁이 표시되지 않는 접근성 문제가 있었기 때문입니다.
📸 Before/After: 해당 없음
♿ Accessibility: 비장애인 및 스크린 리더 사용자 모두 비활성화된 이유("Analyze or open a song first")를 명확히 인지할 수 있도록 개선되었습니다.
---
 apps/desktop/src/features/score/ScoreView.test.tsx | 12 +++++++++++-
 apps/desktop/src/features/score/ScoreView.tsx      |  5 +++--
 2 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/apps/desktop/src/features/score/ScoreView.test.tsx b/apps/desktop/src/features/score/ScoreView.test.tsx
index de4ccb95..4c432e7f 100644
--- a/apps/desktop/src/features/score/ScoreView.test.tsx
+++ b/apps/desktop/src/features/score/ScoreView.test.tsx
@@ -96,7 +96,7 @@ describe("ScoreView", () => {
 
     expect(screen.getByText("Scores attach to the active analysis project.")).toBeInTheDocument();
     expect(screen.getByRole("button", { name: "Add score" })).toBeDisabled();
-    expect(screen.getByRole("button", { name: "Open score: opener.pdf" })).toBeDisabled();
+    expect(screen.getByRole("button", { name: "Open score: opener.pdf" })).toHaveAttribute("aria-disabled", "true");
     expect(screen.getByRole("button", { name: "Remove: opener.pdf" })).toBeDisabled();
 
     fireEvent.click(screen.getByRole("button", { name: "Open score: opener.pdf" }));
@@ -180,6 +180,16 @@ describe("ScoreView", () => {
     });
   });
 
+  it("renders disabled score open buttons with aria-disabled and a tooltip when there is no projectId", () => {
+    const song = makeSong([{ id: SCORE_ID, fileName: "opener.pdf" }]);
+    render(<ScoreView song={song} projectId="" onSongUpdate={vi.fn()} />);
+
+    const button = screen.getByRole("button", { name: "Open score: opener.pdf" });
+    expect(button).toHaveAttribute("aria-disabled", "true");
+    expect(button).toHaveAttribute("title", "scoreNavDisabledHint");
+    expect(button).not.toBeDisabled();
+  });
+
   it("accepts Uint8Array read responses from the bridge", async () => {
     mockInvoke.mockResolvedValueOnce(new Uint8Array([7, 7]));
     const song = makeSong([{ id: SCORE_ID, fileName: "opener.pdf" }]);
diff --git a/apps/desktop/src/features/score/ScoreView.tsx b/apps/desktop/src/features/score/ScoreView.tsx
index 72732450..8437b05e 100644
--- a/apps/desktop/src/features/score/ScoreView.tsx
+++ b/apps/desktop/src/features/score/ScoreView.tsx
@@ -184,10 +184,11 @@ export function ScoreView({ song, projectId, onSongUpdate }: ScoreViewProps) {
                     <button
                       type="button"
                       onClick={projectId ? () => void openAttachment(projectId, attachment) : undefined}
-                      disabled={!projectId}
+                      aria-disabled={!projectId ? true : undefined}
+                      title={!projectId ? t("scoreNavDisabledHint") : undefined}
                       aria-current={selected?.id === attachment.id ? "true" : undefined}
                       aria-label={`${t("scoreOpen")}: ${attachment.fileName}`}
-                      className="flex min-h-10 min-w-0 flex-1 items-center gap-2 text-left text-sm font-semibold text-slate-100 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-cyan-300 disabled:cursor-not-allowed disabled:opacity-60"
+                      className="flex min-h-10 min-w-0 flex-1 items-center gap-2 text-left text-sm font-semibold text-slate-100 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-cyan-300 aria-disabled:cursor-not-allowed aria-disabled:opacity-60"
                     >
                       <FileMusic className="size-4 shrink-0 text-cyan-300" aria-hidden="true" />
                       <span className="truncate">{attachment.fileName}</span>

From 3e343b4a83235231794b0e04d523396cd64e173c Mon Sep 17 00:00:00 2001
From: Seongho Bae <me@seonghobae.me>
Date: Mon, 3 Aug 2026 11:29:51 +0900
Subject: [PATCH 2/4] fix(a11y): expose disabled score reason to assistive tech

---
 apps/desktop/src/features/score/ScoreView.tsx | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/apps/desktop/src/features/score/ScoreView.tsx b/apps/desktop/src/features/score/ScoreView.tsx
index 8437b05e..a5605afb 100644
--- a/apps/desktop/src/features/score/ScoreView.tsx
+++ b/apps/desktop/src/features/score/ScoreView.tsx
@@ -1,4 +1,4 @@
-import { useMemo, useRef, useState } from "react";
+import { useId, useMemo, useRef, useState } from "react";
 import { FileMusic, FilePlus2, Loader2, Trash2 } from "lucide-react";
 import type { RehearsalSong, ScoreAttachment } from "@bandscope/shared-types";
 import { createTranslator, detectPreferredLocale } from "../../i18n";
@@ -39,6 +39,7 @@ function bridgeErrorDetail(error: unknown, fallback: string): string {
  */
 export function ScoreView({ song, projectId, onSongUpdate }: ScoreViewProps) {
   const t = useMemo(() => createTranslator(detectPreferredLocale()), []);
+  const scoreRequiresProjectId = useId();
   const attachments = useMemo(() => song.scoreAttachments ?? [], [song.scoreAttachments]);
   const [selected, setSelected] = useState<ScoreAttachment | null>(null);
   const [pdfBytes, setPdfBytes] = useState<Uint8Array | null>(null);
@@ -149,7 +150,10 @@ export function ScoreView({ song, projectId, onSongUpdate }: ScoreViewProps) {
           </div>
 
           {!projectId && (
-            <p className="rounded-xl border border-amber-300/25 bg-amber-300/10 px-4 py-3 text-sm font-medium text-amber-100">
+            <p
+              id={scoreRequiresProjectId}
+              className="rounded-xl border border-amber-300/25 bg-amber-300/10 px-4 py-3 text-sm font-medium text-amber-100"
+            >
               {t("scoreRequiresProject")}
             </p>
           )}
@@ -185,6 +189,7 @@ export function ScoreView({ song, projectId, onSongUpdate }: ScoreViewProps) {
                       type="button"
                       onClick={projectId ? () => void openAttachment(projectId, attachment) : undefined}
                       aria-disabled={!projectId ? true : undefined}
+                      aria-describedby={!projectId ? scoreRequiresProjectId : undefined}
                       title={!projectId ? t("scoreNavDisabledHint") : undefined}
                       aria-current={selected?.id === attachment.id ? "true" : undefined}
                       aria-label={`${t("scoreOpen")}: ${attachment.fileName}`}

From 93b03bca68190babaed84b8f51a13bec89bb2fd6 Mon Sep 17 00:00:00 2001
From: Seongho Bae <me@seonghobae.me>
Date: Mon, 3 Aug 2026 11:30:19 +0900
Subject: [PATCH 3/4] test(a11y): bind disabled score controls to visible
 reason

---
 .../score/ScoreView.accessibility.test.tsx    | 70 +++++++++++++++++++
 1 file changed, 70 insertions(+)
 create mode 100644 apps/desktop/src/features/score/ScoreView.accessibility.test.tsx

diff --git a/apps/desktop/src/features/score/ScoreView.accessibility.test.tsx b/apps/desktop/src/features/score/ScoreView.accessibility.test.tsx
new file mode 100644
index 00000000..242194fd
--- /dev/null
+++ b/apps/desktop/src/features/score/ScoreView.accessibility.test.tsx
@@ -0,0 +1,70 @@
+"";
+import { render, screen } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import type { RehearsalSong } from "@bandscope/shared-types";
+import { ScoreView } from "./ScoreView";
+
+vi.mock("@tauri-apps/api/core", () => ({
+  invoke: vi.fn()
+}));
+
+vi.mock("./ScoreViewer", () => ({
+  ScoreViewer: () => <div data-testid="score-viewer" />
+}));
+
+vi.mock("../../i18n", () => ({
+  createTranslator: () => (key: string) =>
+    ({
+      scoreViewTitle: "Score",
+      scoreViewSubtitle: "Attach validated PDF scores to the current song.",
+      scoreListTitle: "Attached scores",
+      scoreAttach: "Add score",
+      scoreRemove: "Remove",
+      scoreOpen: "Open score",
+      scoreRequiresProject: "Scores attach to the active analysis project.",
+      scoreNavDisabledHint: "Analyze or open a song first"
+    })[key] ?? key,
+  detectPreferredLocale: () => "en"
+}));
+
+const song = {
+  id: "song-a11y",
+  title: "Accessible Set",
+  sections: [],
+  exportSummary: { format: "cue-sheet", headline: "", focusSections: [] },
+  scoreAttachments: [
+    {
+      id: "score-a11y",
+      fileName: "opener.pdf"
+    }
+  ]
+} as RehearsalSong;
+
+describe("ScoreView disabled-score accessibility", () => {
+  it("references the visible disabled reason from each unavailable score button", () => {
+    render(<ScoreView song={song} projectId={null} onSongUpdate={vi.fn()} />);
+
+    const reason = screen.getByText("Scores attach to the active analysis project.");
+    const openButton = screen.getByRole("button", { name: "Open score: opener.pdf" });
+    const descriptionId = openButton.getAttribute("aria-describedby");
+
+    expect(openButton).toHaveAttribute("aria-disabled", "true");
+    expect(openButton).toHaveAttribute("title", "Analyze or open a song first");
+    expect(descriptionId).toBe(reason.id);
+    expect(descriptionId).not.toBe("");
+    expect(document.getElementById(descriptionId ?? "")).toBe(reason);
+  });
+
+  it("removes disabled-only semantics when a project workspace is available", () => {
+    render(<ScoreView song={song} projectId="project-a11y" onSongUpdate={vi.fn()} />);
+
+    const openButton = screen.getByRole("button", { name: "Open score: opener.pdf" });
+
+    expect(openButton).not.toHaveAttribute("aria-disabled");
+    expect(openButton).not.toHaveAttribute("aria-describedby");
+    expect(openButton).not.toHaveAttribute("title");
+    expect(
+      screen.queryByText("Scores attach to the active analysis project.")
+    ).not.toBeInTheDocument();
+  });
+});

From a1fbd1941b5a5b42cb37aa2b1beea325ef63bec1 Mon Sep 17 00:00:00 2001
From: Seongho Bae <me@seonghobae.me>
Date: Mon, 3 Aug 2026 11:30:51 +0900
Subject: [PATCH 4/4] test(a11y): clean accessibility regression fixture

---
 apps/desktop/src/features/score/ScoreView.accessibility.test.tsx | 1 -
 1 file changed, 1 deletion(-)

diff --git a/apps/desktop/src/features/score/ScoreView.accessibility.test.tsx b/apps/desktop/src/features/score/ScoreView.accessibility.test.tsx
index 242194fd..b960456c 100644
--- a/apps/desktop/src/features/score/ScoreView.accessibility.test.tsx
+++ b/apps/desktop/src/features/score/ScoreView.accessibility.test.tsx
@@ -1,4 +1,3 @@
-"";
 import { render, screen } from "@testing-library/react";
 import { describe, expect, it, vi } from "vitest";
 import type { RehearsalSong } from "@bandscope/shared-types";