chore(pr): preserve superseded entry-name UI lane - #756
seonghobae wants to merge 10 commits into
Conversation
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정 - 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`) - `.jules/palette.md` 저널에 학습 내용 기록 - `CHANGELOG.md` 업데이트
|
👋 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 New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
Understand this PR’s impact Explore downstream dependencies and potential security impact with Blast Radius. Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueNo actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (4)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough생성된 디렉토리 항목의 호버 및 키보드 포커스 밑줄 대상을 Changes항목 이름 밑줄 선택자 수정
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)
✅ Passed checks (3 passed)
Full details: Docstring CoverageExplanation 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 💡
🧪 Generate unit tests (beta)
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. Comment |
- `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` 업데이트
- `span:last-child` 대신 `.entry-name` 클래스를 사용하여 숨겨진 텍스트로 밑줄 스타일이 누출되는 문제 수정 - 관련 테스트 코드 업데이트 (`GeneratedIndexReadabilityTest.kt`) - `.jules/palette.md` 저널에 학습 내용 기록 - `CHANGELOG.md` 업데이트
현재 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-nameproduction/test delta를 보유하고 있으며, BiDi display-integrity 작업과의 통합 순서 및 실제 browser/keyboard acceptance까지 명시적으로 관리하고 있습니다. #769의 현재 branch가 추가로 전진했지만 직전 authority06d37d4...대비 tree diff 없는 continuation이므로 UI semantic ownership은 유지됩니다.따라서 generated history는 지우지 않고, 원래 head
802acdec...를 부모로 protectedmaster@728f0f33323e43573d6664209891099502827d5d의 tree를 사용하는 ordinary descendant8de4684...로 전진시켰습니다. branch update는force=false였고, 현재 protected-base 대비 effective diff는 0 files입니다.이 PR을 단순 duplicate/zero-diff라는 이유로 Close하지 않습니다. #769 또는 verified successor가
.entry-namesource 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은 금지합니다.