Skip to content

Stabilize comment geometry recovery test - #142

Merged
MaggieAppleton merged 1 commit into
mainfrom
maggie/stabilize-comment-geometry-ci
Aug 28, 2026
Merged

Stabilize comment geometry recovery test#142
MaggieAppleton merged 1 commit into
mainfrom
maggie/stabilize-comment-geometry-ci

Conversation

@MaggieAppleton

Copy link
Copy Markdown
Collaborator

Summary

  • select a bounded phrase in the compact comment geometry recovery test
  • keep the tall document while guaranteeing the widened viewport has valid marker geometry

Root cause

The test selected the entire tall paragraph. On Linux at 430px, no 44px marker position could avoid the selected prose, so the product correctly kept the marker beyond the passage and the test timed out.

Verification

  • bun run ci
  • bun run types
  • focused Playwright test
  • focused Playwright test repeated 10 times in parallel

@MaggieAppleton
MaggieAppleton merged commit 950d85f into main Aug 28, 2026
3 checks passed
@MaggieAppleton
MaggieAppleton deleted the maggie/stabilize-comment-geometry-ci branch August 28, 2026 16:31
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