Skip to content

Add regression E2E test to verify PWA pull-to-refresh is disabled - #19

Closed
martinezharo wants to merge 1 commit into
mainfrom
codex/remove-refresh-gesture-in-mobile-pwa
Closed

Add regression E2E test to verify PWA pull-to-refresh is disabled#19
martinezharo wants to merge 1 commit into
mainfrom
codex/remove-refresh-gesture-in-mobile-pwa

Conversation

@martinezharo

Copy link
Copy Markdown
Owner

Motivation

  • Prevent Chromium's pull-to-refresh gesture from interfering with the installed mobile PWA by ensuring overscroll-behavior-y: none applies when running in display-mode: standalone.
  • Add regression coverage so changes to the stylesheet can't accidentally re-enable the gesture.

Description

  • Add a Playwright E2E test in e2e/smoke.spec.ts that inspects document.styleSheets and verifies a @media (display-mode: standalone) rule sets overscroll-behavior-y: none for html, body.
  • Keep the existing stylesheet src/app/pwa.css (which contains the @media (display-mode: standalone) rule) and rely on the test to detect regressions.

Testing

  • Ran the focused Playwright test: pnpm exec playwright test e2e/smoke.spec.ts --project=chromium --grep 'installed PWA disables', which passed.
  • Ran ESLint on the modified test file with pnpm exec eslint e2e/smoke.spec.ts, which returned clean results for the change.
  • Ran the full unit test suite with pnpm test, which completed successfully (13 files, 146 tests passed).

Codex Task

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
klipcode a86e89f Commit Preview URL

Branch Preview URL
Jul 28 2026, 08:48 AM

@martinezharo
martinezharo deleted the codex/remove-refresh-gesture-in-mobile-pwa branch July 31, 2026 09:21
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.

1 participant