🎨 Palette: 악보 뷰어 아이콘 버튼 툴팁 및 접근성 개선

[seonghobae]

Draft owner/consolidation status

Protected base is develop@314ddeae7b775a4957594b599358c8255617eb2e; exact head is b794906f152503459fdb5f156dcf09a6305af350. This PR is intentionally Open / Draft / mergeable.

The two-file delta adds shared Base UI tooltips to ScoreViewer zoom/page icon controls and keeps first/last page controls keyboard-focusable with aria-disabled plus activation guards. That is a valid buyer-facing accessibility direction, but the generated PR was prematurely Ready: all exact-head repository workflows were still queued and there was no formal current-head review submission.

Single-writer finding

Active Draft #1145 already owns ScoreViewer.tsx, ScoreViewer.test.tsx, ScoreView.tsx, ScoreView.test.tsx, pdfjs.ts and pdfjs.test.ts for the bounded Score/PDF semantic-identifier/ACL migration (data → scorePdfBytes, distinct viewer/render cancellation authorities, semantic viewer state names and vendor-boundary translation). #1233 currently edits two of those same owner files directly from protected develop.

Do not merge #1233 independently while that overlapping source owner remains unresolved. The tooltip interaction delta must be adopted/adapted through one canonical Score/PDF owner using ordinary descendant/non-force consolidation; neither branch's predecessor checks/reviews transfer to the consolidated head. The current #1233 source is preserved until that succession is verified rather than closed or force-rewritten.

Accessibility / evidence boundary

The current source-level contract proves only that the previous/next controls expose aria-disabled and the click handlers fail closed at page boundaries. Tooltip copy currently repeats the action label; it does not independently establish a boundary-reason description. jsdom/source evidence is not browser proof of hover/focus persistence, Escape dismissal, pointer/touch hit geometry, 400% zoom/reflow, forced-colors, or Narrator/VoiceOver behavior.

No exact-head workflow is promoted from queued/pending to success, and CodeRabbit's no-actionable-comment summary is not a qualifying independent approval. Keep Draft until source ownership is consolidated and one unchanged final head/base has applicable CI/build/security/SAST/SBOM/coverage gates terminal-success, zero valid findings, current-head browser/accessibility evidence for the material interaction, and qualifying independent non-author approval.

No self-approval, force-push, destructive rebase, gate weakening, synthetic status, no-op freshness commit, or duplicate ScoreViewer source writer.

[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.

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: c83f8807-bb85-4a75-8d83-5bd4aed5ae67

📥 Commits

Reviewing files that changed from the base of the PR and between 314ddea and b794906.

📒 Files selected for processing (2)

• apps/desktop/src/features/score/ScoreViewer.test.tsx
• apps/desktop/src/features/score/ScoreViewer.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

ScoreViewer의 확대/축소 및 페이지 이동 컨트롤이 Tooltip 기반 트리거를 사용하도록 변경되었습니다. 페이지 경계에서는 aria-disabled를 설정하고 클릭을 차단합니다. 관련 테스트는 접근성 상태와 페이지 유지 동작을 검증합니다.

Changes

ScoreViewer 컨트롤

Layer / File(s)	Summary
Tooltip 컨트롤 및 페이지 이동 제어 apps/desktop/src/features/score/ScoreViewer.tsx	확대/축소 및 이전/다음 페이지 컨트롤을 Tooltip 기반 트리거로 변경했습니다. 페이지 경계에서 aria-disabled를 설정하고 클릭 이벤트를 차단합니다.
컨트롤 상태 테스트 apps/desktop/src/features/score/ScoreViewer.test.tsx	페이지 경계의 aria-disabled 상태를 확인합니다. 비활성 이전/다음 버튼을 클릭해도 현재 페이지가 변경되지 않는지 확인합니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to b7949

The updated controls retain accessible names and page-boundary interaction behavior without a supported accessibility regression. The change is ready to merge.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 악보 뷰어 아이콘 버튼의 툴팁 추가와 접근성 개선이라는 주요 변경 사항을 정확하고 간결하게 설명합니다.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}