Skip to content

test(e2e): wide-tables 的粘贴改成替换整篇——光标落点不再看版面脸色 - #242

Closed
oratis wants to merge 1 commit into
mainfrom
fix/e2e-wide-tables-caret
Closed

test(e2e): wide-tables 的粘贴改成替换整篇——光标落点不再看版面脸色#242
oratis wants to merge 1 commit into
mainfrom
fix/e2e-wide-tables-caret

Conversation

@oratis

@oratis oratis commented Aug 23, 2026

Copy link
Copy Markdown
Owner

独立的小修,不在标签页那组叠放 PR(#238#239#240)里,但它们每一个的 E2E 都因为这条红着。

Summary

  • e2e/wide-tables.spec.ts 在 main 上就时红时绿:main 的 head(703402d)那次 CI 也红,同一条断言,量到 685 而不是 720。本地 --workers 4 --repeat-each 3 跑 27 次复现 8 次。
  • 根因:pasteMarkdown 点编辑区中心放光标,落点取决于版面(字体加载、mermaid 异步渲染后的高度变化),落进列表项或引用块,表格就嵌套在里面,量到的是容器宽度而不是正文列。设计 09 §5 那次"等图渲染完再点"只压住了一种时序。
  • 修法:粘贴前 ⌘A。点击只负责聚焦,光标落在哪都无所谓——整篇被替换,表格必然是顶层块。

How to verify

pnpm exec playwright test e2e/wide-tables.spec.ts --workers 4 --repeat-each 4

改前(main)大概率能看到几条 685 vs 720;改后 36 次全绿。全套 e2e 29 条也全绿。

Checks

  • pnpm lint clean(e2e 目录)
  • pnpm test:e2e green(29)
  • 其余项不适用——只动一个 e2e helper

🤖 Generated with Claude Code

wide-tables.spec.ts 在 main 上就时红时绿(703402d 那次 CI 也红):
量到 685 而不是 720。本地 4 workers 跑 27 次复现 8 次。

根因:pasteMarkdown 点编辑区**中心**放光标,落点取决于版面——字体
加载、mermaid 异步渲染后的高度变化——落进列表项或引用块,表格就嵌
套在里面,量到的是容器宽度(685)而不是正文列(720)。设计 09 §5 那
次"等图渲染完再点"只压住了一种时序,CI 上还有别的。

修法:粘贴前 ⌘A。点击只负责聚焦,光标落在哪都无所谓——整篇被替换,
表格必然是顶层块。改后 4 workers × 4 轮 36 次全绿,全套 e2e 29 条全绿。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@oratis

oratis commented Aug 23, 2026

Copy link
Copy Markdown
Owner Author

重复了——#241 已经合了,同一根因同一修法(源码模式整篇替换更彻底)。关掉,把叠放的三个 PR rebase 到新 main 上让 E2E 变绿。

@oratis oratis closed this Aug 23, 2026
@oratis
oratis deleted the fix/e2e-wide-tables-caret branch August 23, 2026 16:46
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