Skip to content

fix(timeline): hitTest 过滤隐藏轨道,隐藏轨片段不可点选/拖拽/框选 (#146)#152

Open
cuic19053-hue wants to merge 1 commit into
appergb:mainfrom
cuic19053-hue:fix-146-hit-test-hidden-track
Open

fix(timeline): hitTest 过滤隐藏轨道,隐藏轨片段不可点选/拖拽/框选 (#146)#152
cuic19053-hue wants to merge 1 commit into
appergb:mainfrom
cuic19053-hue:fix-146-hit-test-hidden-track

Conversation

@cuic19053-hue

Copy link
Copy Markdown
Contributor

fixes #146

问题

上游一致性审计发现:\hitTestClip\(web/src/components/timeline/hitTest.ts)未过滤 \ rack.hidden\,隐藏轨道上的片段仍可被命中拖拽,导致混乱。

修复

在命中测试与框选两个入口均跳过 \ rack.hidden\ 的轨道:

  1. \hitTestClip\:\onPointerDown\ 点选/拖拽命中前 \if (track.hidden) continue\
  2. \clipsInRect\:marquee 框选前 \if (timeline.tracks[ti].hidden) continue\

两处保持一致,确保隐藏轨的片段既不能被单点拖拽,也不能被框选批量选中。

与上游一致性

对齐上游隐藏轨道行为:hidden track 上的片段不参与命中测试。

验证

  • \pnpm run build\(tsc -b + vite build)✅ 通过
  • 改动范围:1 文件 +4 行,纯防御性 \continue\,无回归风险

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.

[timeline][bug] 隐藏轨道上的片段仍可被拖拽(hitTest 未过滤 track.hidden)

1 participant