Conversation
|
@tevinch is attempting to deploy a commit to the afc163's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No 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 (2)
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review. WalkthroughNotificationList 在悬停期间监听文档级鼠标移动,并检查事件路径和列表的 Changes通知悬停恢复
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~12 minutes Change: Bug fix · Severity of issue fixed: Medium Merge Risk: ⚪ Minimal · up to The hover-recovery change is mergeable after normal checks. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #413 +/- ##
==========================================
+ Coverage 99.20% 99.22% +0.02%
==========================================
Files 12 12
Lines 376 387 +11
Branches 102 104 +2
==========================================
+ Hits 373 384 +11
Misses 3 3 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Closing the bottom notice in an expanded stack can remove the hovered DOM node without React receiving
mouseleave. The remaining notices stay expanded with auto-close timers paused even after the pointer moves outside.This adds movement recovery while the list is hovered. The event path identifies movement within the list, with native
:hoveras a fallback for closed shadow roots that hide their internals from document events. Moving within the list keeps timers paused; moving outside resumes the remaining duration. Persistent notices and the configured stack threshold are preserved.Fixes ant-design/ant-design#59412, Close #383, which addressed the same interaction before the notification list refactor.
Validation:
npm test: 57 tests passed, including regressions for 4→3 and 5→4 notices, a custom threshold, repeated use, remaining duration, persistent content, stopped propagation, and nested shadow containers.npm run tscandnpm run compilepassed.npm run lint: no errors; 10 existing warnings in unchanged files.Summary by CodeRabbit