Skip to content

fix(sidebar): scope scroll hover updates - #2

Merged
kilockok merged 1 commit into
mainfrom
agent/address-sidebar-hover-review
Aug 14, 2026
Merged

fix(sidebar): scope scroll hover updates#2
kilockok merged 1 commit into
mainfrom
agent/address-sidebar-hover-review

Conversation

@kilockok

Copy link
Copy Markdown
Owner

修改

  • P1:移除对 scrollend 和持续递归 RAF 的依赖;每批 scroll 事件只合并为一个下一帧命中测试,执行后立即停止。
  • P2:仅在 @media (hover: hover) 下抑制 Chromium 原生 :hover,保留触屏 :active::before 按压反馈。

验证

  • ESLint:通过
  • pnpm type-check:通过
  • pnpm build-only:通过
  • 一次性 RAF 行为断言:通过,确认事件合帧且 RAF 不会递归
  • node scripts/check-ui-contract.mjs:仓库现有 Web 基线失配导致 651 条无关错误;本次差异未新增原始颜色、尺寸或阴影值

⚠️ No human QA — this PR has not been verified by a human yet. Remove this line once a human confirms the happy path.

@kilockok
kilockok merged commit 401ceab into main Aug 14, 2026
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