preserve(score): zero-delta ScoreViewer tooltip provenance pending #731

[seonghobae]

Preservation / succession status

이 PR은 독립 ScoreViewer accessibility source owner가 아닙니다.

• Protected base: develop@314ddeae7b775a4957594b599358c8255617eb2e
• Exact current head: e9ed0d39db53199e6680fb90db4a0b9763d2b190
• Prior authority head: 4d1ad7d64a88013df3a6291143700a7d5191ebfb
• Open / Draft / mergeable
• Fresh protected-base diff: 0 files
• 4d1ad7d... → e9ed0d3...: four ordinary descendant commits, zero file delta
• Canonical Score accessibility owner: 🎨 Palette: 향상된 접근성을 위해 아이콘 버튼에 툴팁 및 aria-disabled 적용 #731 exact 36719ab3b805cc64ce6be70eea35cacb3ce6ad94
• Score/PDF semantic naming owner: refactor(naming): use semantic temporal and score identifiers #1145 exact 716b3df51a1e4b18a759f2ac260d86cf627532f5

Generated continuation disposition

Earlier preservation head 1db8197ba1f8104760a60430eab991f635b32686 retained shared Tooltip use plus focusable aria-disabled pagination with activation guards. Generated descendant 4d1ad7d64a88013df3a6291143700a7d5191ebfb then removed that duplicate source: it returned zoom/page controls to plain Button rendering, changed page boundaries back to native disabled, and weakened the local tests accordingly.

Current e9ed0d3... advances four commits beyond that authority head but fresh compare reports no file delta, and protected-base diff remains zero. This is source-neutral movement, not a new ScoreViewer repair. Do not create a wake/restack commit or promote predecessor checks/reviews merely because the tree is unchanged.

Because canonical #731 has already adopted and strengthened the valid reusable-tooltip/focusable-boundary semantics, restoring the weaker duplicate source here would create another Score accessibility writer rather than preserve unique product truth.

#731 retains shared Tooltip / TooltipTrigger / TooltipContent, focusable aria-disabled, guarded activation, reason-specific EN/KO copy, and persistent aria-describedby boundary reasons that remain available when the visual tooltip is dismissed. The valid semantic delta therefore remains on the canonical owner, not this zero-delta branch.

PR-0 rule

Zero file delta does not justify premature Close while #731 is still an unmerged Draft and still requires reconciliation with #1145's overlapping Score/PDF naming tree. Keep this PR open as provenance until a verified #731/#1145 successor preserves the valid accessibility contract, obtains fresh exact-head gates plus qualifying independent non-author approval, and reaches protected ancestry.

No source-neutral wake commit, duplicate Tooltip implementation, self-approval, force-push, destructive rebase, gate weakening, synthetic status, blind rerun, or predecessor-evidence transfer is used.

UI Delivery Gate: FAIL. The canonical source contract exists on #731, but packaged Electron/browser pointer/touch/keyboard, 400% zoom/reflow, forced-colors, Narrator/VoiceOver and KO/EN/JA/ZH/VI/ES/DE/FR acceptance remain unproven.

[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	제목에 ScoreViewer 툴팁이 포함되어 변경 사항의 핵심을 일부 설명합니다. 다만 ‘zero-delta’, ‘provenance pending #731’은 변경 목적을 명확히 전달하지 않습니다.

✨ 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.}

[seonghobae]

Current authority update: live head is 6d86c7de511241c45b6d1bc7c4dd492ab074eacb, one descendant after body-listed 4d1ad7d64a88013df3a6291143700a7d5191ebfb. Fresh compare shows zero file delta and the same protected tree e30d0e0cd58aa67a5dad7254948717d796d0ce36. This is source-neutral authority movement, not product/accessibility repair progress, so no new commit is warranted. #731/#1145 remain the canonical Score accessibility/naming succession path; this PR stays Open/Draft zero-delta provenance until verified protected succession.

[google-labs-jules]

Current authority update: live head is 6d86c7de511241c45b6d1bc7c4dd492ab074eacb, one descendant after body-listed 4d1ad7d64a88013df3a6291143700a7d5191ebfb. Fresh compare shows zero file delta and the same protected tree e30d0e0cd58aa67a5dad7254948717d796d0ce36. This is source-neutral authority movement, not product/accessibility repair progress, so no new commit is warranted. #731/#1145 remain the canonical Score accessibility/naming succession path; this PR stays Open/Draft zero-delta provenance until verified protected succession.

Understood. Acknowledging that this work is now obsolete and stopping work on this task.

[seonghobae]

Current-head authority refresh: branch ref is now 88756b5806047d2004f4df1567e514c745b2613e, two ordinary commits ahead of body-recorded 4d1ad7d64a88013df3a6291143700a7d5191ebfb. Fresh compare reports files=[]; zero-delta ScoreViewer provenance and canonical #731/#1145 ownership are unchanged, so semantic repair progress from this movement is 0. Keep Open / Draft; do not create a wake commit or promote predecessor evidence.

[google-labs-jules]

Current-head authority refresh: branch ref is now 88756b5806047d2004f4df1567e514c745b2613e, two ordinary commits ahead of body-recorded 4d1ad7d64a88013df3a6291143700a7d5191ebfb. Fresh compare reports files=[]; zero-delta ScoreViewer provenance and canonical #731/#1145 ownership are unchanged, so semantic repair progress from this movement is 0. Keep Open / Draft; do not create a wake commit or promote predecessor evidence.

Understood. Acknowledging that this work is now obsolete and stopping work on this task.

[seonghobae]

Fresh head authority: branch ref is now 040425a01cd5c68aa9597d81a52d634ca5f479be. Compare from body-listed 4d1ad7d64a88013df3a6291143700a7d5191ebfb is ahead by 3 commits with files=[]. This is source-neutral movement; the zero-delta ScoreViewer preservation tree is unchanged and repair progress is 0. No wake/restack commit, predecessor-evidence promotion, or duplicate accessibility source is justified.

[google-labs-jules]

Fresh head authority: branch ref is now 040425a01cd5c68aa9597d81a52d634ca5f479be. Compare from body-listed 4d1ad7d64a88013df3a6291143700a7d5191ebfb is ahead by 3 commits with files=[]. This is source-neutral movement; the zero-delta ScoreViewer preservation tree is unchanged and repair progress is 0. No wake/restack commit, predecessor-evidence promotion, or duplicate accessibility source is justified.

Understood. Acknowledging that this work is now obsolete and stopping work on this task.

[seonghobae]

Closing as a duplicate of #731, which addresses the same issue. Thanks!

[google-labs-jules]

Closing as a duplicate of #731, which addresses the same issue. Thanks!

Understood. Acknowledging that this work is now obsolete and stopping work on this task.