🎨 Palette: [UX improvement] 악보 뷰어 아이콘 버튼의 툴팁 및 접근성(aria-disabled) 향상

[seonghobae]

💡 What:
ScoreViewer 내의 아이콘 버튼(이전 페이지, 다음 페이지, 확대, 축소)들에 대해 호버 시 정보를 제공할 수 있도록 title 속성을 추가하고, 비활성화 처리 시 네이티브 disabled 속성 대신 aria-disabled="true"를 사용하도록 수정했습니다. 관련된 클릭 핸들러들에도 비활성화 조건 시 조기 반환(early return)하여 클릭 이벤트가 발생하지 않도록 조치했습니다.

🎯 Why:
네이티브 disabled 요소는 키보드 포커스(Tab) 이동 순서에서 제외되며 툴팁 동작 또한 방해합니다. aria-disabled 속성을 사용하여 포커스를 유지하게 함으로써, 스크린 리더 사용자와 키보드 이동 사용자 모두가 버튼의 존재와 상태(비활성화 여부)를 쉽게 인지할 수 있도록 접근성과 UX를 개선하기 위함입니다.

📸 Before/After:

• Before: 아이콘 전용 버튼에 호버 시 툴팁이 없고, 비활성화 상태에서 Tab 포커스가 건너뛰어짐.
• After: 아이콘 버튼 호버 시 툴팁 텍스트 노출. 페이지 한계 도달 또는 확대/축소 한계치 도달 시 버튼이 aria-disabled 상태로 변환되며 Tab 포커스는 유지됨.

♿ Accessibility:

• 모든 조작용 아이콘 버튼이 키보드 포커스 탭 순서(tab order)에 정상적으로 포함됩니다.
• 아이콘만 있는 버튼에 title 속성을 부여하여 마우스 사용자 및 스크린리더에게 명확한 맥락(Tooltip)을 제공합니다.
• disabled 속성을 제거함으로써 스크린 리더가 요소의 존재 자체를 무시하지 않고, 비활성화된 이유를 명확하게 알 수 있도록 지원합니다.

---

PR created automatically by Jules for task 11209809002286895946 started by @seonghobae

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[opencode-agent]

Pull request overview

OpenCode cannot approve yet because required coverage evidence did not pass.

Review outcome

1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence

• Problem: The required coverage-evidence job result was failure, so OpenCode cannot establish approval sufficiency for this head.

• Root cause: Automated approval is only valid when the same-head coverage-evidence job proves supported repository test suites passed and configured docstring gates passed or were advisory, or reports not applicable because no supported source files or package manifests exist. Missing, failed, skipped, unavailable, or unsupported-tooling test evidence is a blocker.

• Fix: Install or configure the repository test/docstring evidence tooling when source files or package manifests exist, rerun the current-head coverage-evidence job, and approve only after it reports success with required evidence or explicit no-source not-applicable evidence.

• Regression test: Keep the approval branch checking needs.coverage-evidence.result == success before posting APPROVE, and publish REQUEST_CHANGES when coverage-evidence blocker states such as cancelled, skipped, failed, unsupported-tooling, or below-100 evidence are present.

• Result: REQUEST_CHANGES

• Reason: coverage-evidence result was failure, so required test/docstring evidence was not proven for current head 37fa72faf06f922e8de83caa093569b7ea8300e2.

• Head SHA: 37fa72faf06f922e8de83caa093569b7ea8300e2

• Workflow run: 29539109582

• Workflow attempt: 1

Coverage evidence

Coverage evidence job did not run or did not publish coverage evidence.

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Changed file (3 files)"]
  S1 --> I1["repository behavior"]
  I1 --> R1["Review risk: Changed file (3 files)"]
  R1 --> V1["required checks"]

Loading

[opencode-agent]

OpenCode Review Overview

• Head SHA: 37fa72faf06f922e8de83caa093569b7ea8300e2
• Workflow run: 29539109582
• Workflow attempt: 1
• Gate result: REQUEST_CHANGES (approval step)

Pull request overview

OpenCode cannot approve yet because required coverage evidence did not pass.

Review outcome

1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence

• Problem: The required coverage-evidence job result was failure, so OpenCode cannot establish approval sufficiency for this head.

• Root cause: Automated approval is only valid when the same-head coverage-evidence job proves supported repository test suites passed and configured docstring gates passed or were advisory, or reports not applicable because no supported source files or package manifests exist. Missing, failed, skipped, unavailable, or unsupported-tooling test evidence is a blocker.

• Fix: Install or configure the repository test/docstring evidence tooling when source files or package manifests exist, rerun the current-head coverage-evidence job, and approve only after it reports success with required evidence or explicit no-source not-applicable evidence.

• Regression test: Keep the approval branch checking needs.coverage-evidence.result == success before posting APPROVE, and publish REQUEST_CHANGES when coverage-evidence blocker states such as cancelled, skipped, failed, unsupported-tooling, or below-100 evidence are present.

• Result: REQUEST_CHANGES

• Reason: coverage-evidence result was failure, so required test/docstring evidence was not proven for current head 37fa72faf06f922e8de83caa093569b7ea8300e2.

• Head SHA: 37fa72faf06f922e8de83caa093569b7ea8300e2

• Workflow run: 29539109582

• Workflow attempt: 1

Coverage evidence

Coverage evidence job did not run or did not publish coverage evidence.

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Changed file (3 files)"]
  S1 --> I1["repository behavior"]
  I1 --> R1["Review risk: Changed file (3 files)"]
  R1 --> V1["required checks"]

Loading