Skip to content

fix(site): make settings header scrollable so Safari PWA can reach back button#23742

Draft
DanielleMaywood wants to merge 1 commit intomainfrom
fix/safari-pwa-keyboard-scroll
Draft

fix(site): make settings header scrollable so Safari PWA can reach back button#23742
DanielleMaywood wants to merge 1 commit intomainfrom
fix/safari-pwa-keyboard-scroll

Conversation

@DanielleMaywood
Copy link
Copy Markdown
Contributor

@DanielleMaywood DanielleMaywood commented Mar 28, 2026

🤖 This PR was written by Coder Agent on behalf of Danielle Maywood 🤖

In Safari standalone (PWA) mode, the virtual keyboard pushes the visual viewport upward without resizing it. When the keyboard dismisses, Safari often fails to restore the scroll position. On the agent settings behavior page, the AgentPageHeader (with the back button) was a sibling above the scroll container (AgentSettingsPageView), making it permanently unreachable once the viewport shifted.

Moves the scroll container from AgentSettingsPageView up to AgentSettingsPage so both the header and settings content scroll as a single unit. The back button is always reachable by scrolling up.

Decision log
  • Considered a focusout + setTimeout + window.scrollTo(0, 0) workaround in useAgentsPWA but rejected it as fragile (magic timeout, browser-behavior dependent).
  • Structural fix preferred: making the header part of the scrollable area is a common mobile pattern and works across all browsers without JS hacks.
  • Scoped to settings page only — analytics page has the same layout pattern but no text inputs to trigger the keyboard.

@github-actions github-actions bot added the community Pull Requests and issues created by the community. label Mar 28, 2026
…ck button

In Safari standalone (PWA) mode, the virtual keyboard pushes the
visual viewport upward without resizing it. When the keyboard
dismisses, Safari often fails to restore the scroll position.

On the agent settings behavior page, the AgentPageHeader (with the
back button) was a sibling above the scroll container, making it
unreachable once the viewport shifted. Move the scroll container up
to AgentSettingsPage so both the header and settings content scroll
as a single unit — the header is always reachable by scrolling up.
@DanielleMaywood DanielleMaywood force-pushed the fix/safari-pwa-keyboard-scroll branch from 2e1a900 to 90a73fe Compare March 28, 2026 01:37
@DanielleMaywood DanielleMaywood changed the title fix: reset scroll position after Safari PWA keyboard dismiss fix(site): make settings header scrollable so Safari PWA can reach back button Mar 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Pull Requests and issues created by the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant