Skip to content

test(playwright): scope drag selection clear to firefox #31301

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ShaneK merged 1 commit into from
Jul 28, 2026
+10 −1
Merged

test(playwright): scope drag selection clear to firefox #31301

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion core/src/utils/test/playwright/drag-element.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,16 @@ export const dragElementBy = async (
// initiates text selection on mouse.down(). In Playwright, this selection
// locks in before the gesture detector fires, blocking pointer events.
// Clearing it here allows the drag gesture to proceed normally.
await page.evaluate(() => window.getSelection()?.removeAllRanges());
//
// This is scoped to Firefox because the extra page.evaluate() is an awaited
// round-trip that adds variable latency between mouse.down() and the first
// mouse.move(). Chromium and WebKit don't need the selection clear, and on
// WebKit that latency perturbs the timing-derived gesture velocity, causing
// the item to settle a few pixels off and flaking screenshot comparisons
// (e.g. item-sliding safe-area tests).
if (page.context().browser()?.browserType().name() === 'firefox') {
await page.evaluate(() => window.getSelection()?.removeAllRanges());
}

// Drag the element.
await moveElement(page, startX, startY, dragByX, dragByY);
Expand Down
Loading