🎨 Palette: 파일 목록 호버(Hover) 시 밑줄 효과 복원

[seonghobae]

Verified successor disposition

The valid defect is real: once a translated .visually-hidden type label became the last child inside each link, span:last-child no longer selected the rendered filename and hover/focus underlining disappeared.

This branch repairs that symptom with span:nth-child(2), but the positional selector remains coupled to child order and its regression only asserts the CSS literal. The generated .jules/palette.md addition also tries to generalize this local DOM shape into repository doctrine.

Canonical #599 now fully carries the valid behavior on current protected master with a stronger semantic contract. Exact successor head b811d4721ab3940774e88d1ed73bbfd89137d3a5 adds .entry-name only to visible parent/file-name spans, targets that class for hover, focus-visible and active states, rejects span:last-child, verifies the real rendered row keeps .icon and .visually-hidden outside the target, restores protected Palette doctrine, and has been non-force restacked so it is behind_by=0 against master@728f0f33323e43573d6664209891099502827d5d.

#599 is still Draft with fresh current-head CI/security/SAST/CodeQL queued; no gate evidence transfers from this branch. No unique valid semantic/test/contract delta remains here once the positional implementation and repository-wide doctrine are rejected. This close is therefore verified-successor consolidation, not simple PR-count reduction.

[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: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 260731c8-0412-4469-af1f-ba2611c1c82b

📥 Commits

Reviewing files that changed from the base of the PR and between 728f0f3 and 15be83a.

📒 Files selected for processing (3)

• .jules/palette.md
• src/main/kotlin/html4tree/main.kt
• src/test/kotlin/html4tree/GeneratedIndexReadabilityTest.kt

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

---

📝 Walkthrough

Walkthrough

링크의 호버 및 키보드 포커스 밑줄 선택자를 표시 파일명에 맞게 변경했습니다. 관련 테스트를 갱신하고, 숨김 텍스트로 인해 밑줄 대상이 변경되는 접근성 문제를 문서화했습니다.

Changes

링크 밑줄 접근성 수정

Layer / File(s)	Summary
표시 파일명 밑줄 선택자 및 검증 .jules/palette.md, src/main/kotlin/html4tree/main.kt, src/test/kotlin/html4tree/GeneratedIndexReadabilityTest.kt	호버 및 포커스 밑줄 대상을 span:last-child에서 span:nth-child(2)로 변경했습니다. 관련 테스트와 접근성 설명을 갱신했습니다.

Estimated code review effort: 1 (매우 단순) | ~3 minutes

Merge Risk: ⚪ Minimal · up to 151f2

Hover and keyboard-focus underlining now targets the visible filename rather than hidden accessibility text. The accessibility label remains intact, and no merge-blocking risk is evident.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (1 skipped: 1 …	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 파일 목록의 호버 시 밑줄 효과를 복원하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
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.

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch palette-hover-underline-fix-17721940346798866804

---

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