🎨 Palette: 키보드 내비게이션 접근성 향상을 위한 포커스 스타일 개선

[seonghobae]

💡 What:
HTML 리포트의 main, .table-wrap, .export-block pre 요소에 대해 기존의 마우스 클릭 시 아웃라인 제거 스타일인 :focus { outline: none; }을 :focus:not(:focus-visible) { outline: none; }으로 변경했습니다.

🎯 Why:
기존에는 마우스 클릭 시 렌더링되는 큰 아웃라인을 제거하기 위해 outline: none;을 사용했지만, 이는 키보드를 사용하여 탐색하는 사용자(예: skip-link를 통해 이동한 사용자)에게도 포커스 링을 숨겨 키보드 접근성을 완전히 파괴하는 문제를 야기했습니다. :focus-visible 가상 클래스를 활용하여 마우스/터치 상호작용에서는 시각적 방해를 없애고, 키보드 내비게이션 시에는 기본 포커스 인디케이터가 명확히 보이도록 수정했습니다.

📸 Before/After:
변경 전: 키보드로 탭 이동 시 요소에 시각적인 포커스 표시가 나타나지 않아 사용자가 현재 위치를 파악하기 어려움.
변경 후: 키보드 이동 시에만 브라우저 기본 포커스 링이 표시되어 명확한 탐색이 가능하며, 마우스 클릭 시에는 불필요한 아웃라인이 나타나지 않음. (Playwright 스크린샷으로 검증 완료)

♿ Accessibility:
키보드 내비게이션 의존 사용자를 위한 핵심 WCAG 2.4.7 (Focus Visible) 지침을 준수하도록 HTML 리포트의 접근성을 향상시켰습니다.

---

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

---

Summary by CodeRabbit

• Bug Fixes

  • Improved focus styling in diagnostic reports so mouse interactions do not unnecessarily display outlines.
  • Preserved clear focus indicators for keyboard navigation across report sections, tables, and export code blocks.

• Tests

  • Updated accessibility checks to verify the revised focus behavior.

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

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 5defa5e0-d952-4cf6-a6ef-9dd902e5d049

📥 Commits

Reviewing files that changed from the base of the PR and between 04d0bc2 and ce6ac56.

📒 Files selected for processing (3)

• python/fast_mlsirm/report.py
• tests/test_report.py
• tests/test_report_focus_contrast.py

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

---

📝 Walkthrough

Walkthrough

The report now suppresses outlines for non-keyboard focus on main, table containers, and export code blocks. Tests update focus selectors and add descriptions for report diagnostics cases.

Changes

Report focus styling

Layer / File(s)	Summary
Focus selector updates python/fast_mlsirm/report.py	Focus rules for main, .table-wrap, and .export-block pre now apply to :focus:not(:focus-visible).
Focus behavior validation tests/test_report.py, tests/test_report_focus_contrast.py	Tests expect non-visible focus selectors and add descriptions for report diagnostics cases.

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

Merge Risk: ⚪ Minimal · up to ce6ac

This localized accessibility improvement preserves focus indicators for keyboard navigation while suppressing unnecessary mouse-focus outlines. No actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: improving focus styles for keyboard navigation accessibility.
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 14 functions across 3 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.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch palette-focus-accessibility-11234531889831780158

---

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

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

[seonghobae]

Current-head review (fe70b02):

• The accessibility change is limited to suppressing mouse-click outlines with :focus:not(:focus-visible) while retaining keyboard-visible focus styles.
• Focused tests: 15 passed. Ruff passed. Interrogate passed at 100% for changed source/tests. git diff --check passed.
• No actionable source finding at this exact head. Please preserve the protected review/check gates.

[seonghobae]

@opencode-agent review @cwl-noema-review review

[google-labs-jules]

Current-head review (fe70b02):

• The accessibility change is limited to suppressing mouse-click outlines with :focus:not(:focus-visible) while retaining keyboard-visible focus styles.
• Focused tests: 15 passed. Ruff passed. Interrogate passed at 100% for changed source/tests. git diff --check passed.
• No actionable source finding at this exact head. Please preserve the protected review/check gates.

Acknowledged.

[seonghobae]

Current-head review (ce6ac56):

• A concurrent Palette update replaced the earlier branch head, so the report test docstrings were reapplied to this exact current HEAD rather than reusing stale evidence.
• Focused tests: 15 passed. Ruff passed. Interrogate passed at 100% for changed source/tests. git diff --check passed.
• No actionable source finding at this exact head. Preserve the protected review/check gates and merge only through normal auto-merge.

[seonghobae]

@opencode-agent review @cwl-noema-review review

[google-labs-jules]

Current-head review (ce6ac56):

• A concurrent Palette update replaced the earlier branch head, so the report test docstrings were reapplied to this exact current HEAD rather than reusing stale evidence.
• Focused tests: 15 passed. Ruff passed. Interrogate passed at 100% for changed source/tests. git diff --check passed.
• No actionable source finding at this exact head. Preserve the protected review/check gates and merge only through normal auto-merge.

Acknowledged.

[seonghobae]

The substantive accessibility delta has been adopted into the existing report-surface owner, PR #1002, instead of keeping two branches mutating the same python/fast_mlsirm/report.py focus contract. On #1002 I first added a regression that rejects blanket .table-wrap:focus / .export-block pre:focus outline suppression, then changed those two selectors to :focus:not(:focus-visible) while retaining the explicit :focus-visible keyboard indicators. #1002 already owned the main focus correction and the generic report-title trust-boundary fix, so consolidating there preserves single-writer scope and avoids duplicate/conflicting report CSS landing units. The unrelated test-docstring churn from this branch was not adopted.