Skip to content

Fix Safari freezes on complex screenshot pages#160

Merged
neonwatty merged 1 commit into
mainfrom
fix-safari-complex-screenshots
May 15, 2026
Merged

Fix Safari freezes on complex screenshot pages#160
neonwatty merged 1 commit into
mainfrom
fix-safari-complex-screenshots

Conversation

@neonwatty

Copy link
Copy Markdown
Collaborator

Summary

  • disable full-page and area capture at the lower complex-DOM threshold for Safari
  • keep Chromium and other browsers on the existing 10k full-page cutoff
  • align submitted metadata with the same full-page-disabled decision used by the widget UI
  • add unit and E2E regression coverage for Safari complex-page capture options

Fixes #158

Verification

  • npm run typecheck
  • npm run lint (existing warnings only)
  • npm run format:check
  • npm test
  • npm test -- test/cropScreenshot.test.ts
  • npx playwright test e2e/widget.spec.ts --grep "Safari pages|very complex pages without native viewport capture|auto mode skips full-page capture"

@neonwatty neonwatty added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit 07b4053 May 15, 2026
6 checks passed
@neonwatty neonwatty deleted the fix-safari-complex-screenshots branch May 15, 2026 00:52
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.33.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Freezing of system.

1 participant