style(anchor): stabilize marker positioning during panel expansion - #409
style(anchor): stabilize marker positioning during panel expansion#409SonyLeo wants to merge 2 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe anchor components now use shared CSS custom properties for surface padding, item padding, tooltip gaps, and marker positioning. List spacing and expanded or collapsed item layouts use the updated values. ChangesAnchor layout
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This updates Anchor spacing and marker positioning, including collapsed-item hit-area handling. No unresolved merge-readiness risk remains. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 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. A rabbit hops where anchors align Comment |
📦 Package Previewpnpm add https://pkg.pr.new/@opentiny/tiny-robot@286737f pnpm add https://pkg.pr.new/@opentiny/tiny-robot-kit@286737f pnpm add https://pkg.pr.new/@opentiny/tiny-robot-svgs@286737f commit: 286737f |
gene9831
left a comment
There was a problem hiding this comment.
发现 1 个需要修正的交互回归:收起态目录项的按钮命中区域由原先约 30px 缩小到 16px。具体复现和修改建议见行内评论。
已在 PR Preview 验证左右 placement、hover/manual 收起态以及 360px 视口;marker 定位稳定,未发现额外横向溢出。当前 CI build、e2e 和 package preview 均通过,但仓库没有 Anchor 专项测试覆盖该命中区域。

变更说明
验证
vue-tsc类型检查通过截图对比
修改前
修改后
Summary by CodeRabbit
Style