Skip to content

chore(pr): preserve superseded entry-name UI lane - #756

Draft
seonghobae wants to merge 10 commits into
masterfrom
palette/hover-underline-fix-16133561947784912599
Draft

seonghobae wants to merge 10 commits into
masterfrom
palette/hover-underline-fix-16133561947784912599

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

현재 exact head: 8de468447e797466258cf7e73cd48c87781e5855. Draft preservation lane입니다.

이 PR의 유효한 제품 delta는 visible filename span에 .entry-name을 부여하고 hover/focus underline selector를 span:last-child에서 .entry-name으로 좁히는 것, 그리고 그 selector를 고정하는 GeneratedIndexReadabilityTest입니다. 원래 branch에는 해당 변경의 CHANGELOG와 retrospective .jules/palette.md도 포함돼 있었습니다.

현재 더 강한 sibling #769이 같은 .entry-name production/test delta를 보유하고 있으며, BiDi display-integrity 작업과의 통합 순서 및 실제 browser/keyboard acceptance까지 명시적으로 관리하고 있습니다. #769의 현재 branch가 추가로 전진했지만 직전 authority 06d37d4... 대비 tree diff 없는 continuation이므로 UI semantic ownership은 유지됩니다.

따라서 generated history는 지우지 않고, 원래 head 802acdec...를 부모로 protected master@728f0f33323e43573d6664209891099502827d5d의 tree를 사용하는 ordinary descendant 8de4684...로 전진시켰습니다. branch update는 force=false였고, 현재 protected-base 대비 effective diff는 0 files입니다.

이 PR을 단순 duplicate/zero-diff라는 이유로 Close하지 않습니다. #769 또는 verified successor가 .entry-name source delta, selector regression, code-current release note/CHANGELOG 판단, 그리고 keyboard focus·visible underline·visually-hidden label 비간섭에 대한 current-head browser/E2E evidence를 정상 protected ancestry에 완전히 승계한 뒤에만 PR-0을 판단합니다.

Delivery Gate: 의도성 PASS / 콘텐츠 적합성 PASS / 기능 완전성 PARTIAL / 복원력 PARTIAL / 증거성 FAIL / 고유성 N/A. 현재 lane 자체는 preservation이므로 UI 완료를 주장하지 않습니다. force push, destructive rebase, source-neutral rerun, predecessor GREEN 승계, gate weakening은 금지합니다.

- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
@google-labs-jules

Copy link
Copy Markdown

👋 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

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

Understand this PR’s impact

Explore downstream dependencies and potential security impact with Blast Radius.

View blast radius →

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: 942be730-514a-4b55-9473-14f2654c1d6b

📥 Commits

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

📒 Files selected for processing (4)
  • .jules/palette.md
  • CHANGELOG.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

생성된 디렉토리 항목의 호버 및 키보드 포커스 밑줄 대상을 span:last-child에서 .entry-name으로 변경했습니다. 파일 이름 스팬에 클래스를 추가하고 관련 테스트와 변경 기록을 갱신했습니다.

Changes

항목 이름 밑줄 선택자 수정

Layer / File(s) Summary
항목 이름 선택자와 마크업 갱신
src/main/kotlin/html4tree/main.kt, src/test/kotlin/html4tree/GeneratedIndexReadabilityTest.kt, CHANGELOG.md, .jules/palette.md
CSS 선택자가 .entry-name을 대상으로 변경되었습니다. 생성되는 파일 이름 스팬에 .entry-name 클래스가 추가되었습니다. 테스트와 변경 기록이 새 선택자를 반영합니다.

Priority: ⬇️ Low

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

Change: Bug fix · Severity of issue fixed: Low

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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 2 functions across 2 files. (2 skipped: 2 … Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive 제목은 entry-name UI와 관련되어 변경 사항의 일부를 반영합니다. 그러나 supersededUI lane이라는 표현이 모호하여 .entry-name 선택자로 밑줄 스타일을 수정한 핵심 변경을 명확하게 설명하지 못합니다. 제목을 .entry-name 선택자를 사용해 파일 및 디렉터리 이름의 호버/포커스 밑줄 표시를 수정한다는 내용으로 변경하십시오.
✅ Passed checks (3 passed)
Check name Status Explanation
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.
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 2 functions across 2 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches 💡 1
📝 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

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

seonghobae and others added 8 commits September 22, 2026 04:02
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
@seonghobae
seonghobae marked this pull request as draft September 23, 2026 19:12
@seonghobae seonghobae changed the title 🎨 Palette: [파일 이름 호버 밑줄 표시 버그 수정 및 개선] chore(pr): preserve superseded entry-name UI lane Sep 23, 2026
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정
- 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`)
- `.jules/palette.md` 저널에 학습 내용 기록
- `CHANGELOG.md` 업데이트
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant