Skip to content

fix(vtable): enlarge fill handle hit area - #5322

Merged
fangsmile merged 2 commits into
developfrom
fix/issue-4656-fill-handle-hit-area
Sep 20, 2026
Merged

fangsmile merged 2 commits into
developfrom
fix/issue-4656-fill-handle-hit-area

Conversation

@fangsmile

@fangsmile fangsmile commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • enlarge the fill handle pointer hit target from 18px to 24px while keeping the visible handle at 6px
  • add focused boundary coverage for the expanded hit area and unchanged visual size
  • add an issue-specific demo and Bugserver Photo case

Test plan

  • rushx test --runInBand __tests__/fill-handle-hit-area.test.ts
  • rushx compile
  • ESLint for changed TypeScript files
  • browser demo verification: hit at 11px, miss at 13px, visual handle 6x6
  • pre-push rush test --only tag:package
  • Bugserver reproduction before the fix: red FAIL state
  • Bugserver branch build 1.0.0.4549: green PASS (hit@11=true, hit@13=false, visual=6x6)

Bugserver case: https://bugserver.cn.goofy.app/case?product=VTable&fileid=6aaf3468168db9005cc382c1&version=1.0.0.4549

Closes #4656

🤖 Generated with Claude Code

Make the fill handle easier to grab without changing its visual size.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
Document why the pointer target intentionally exceeds the visible handle.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
@fangsmile
fangsmile merged commit abee653 into develop Sep 20, 2026
7 of 8 checks passed
@fangsmile
fangsmile deleted the fix/issue-4656-fill-handle-hit-area branch September 20, 2026 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] 填充柄拖动不灵敏,鼠标悬停变加号 的热区太小,导致拖动失效

1 participant