Skip to content

Fix settings page scrolling on short/mobile viewports#98

Merged
wsxiaolin merged 1 commit into
mainfrom
codex-adoten
Jun 7, 2026
Merged

Fix settings page scrolling on short/mobile viewports#98
wsxiaolin merged 1 commit into
mainfrom
codex-adoten

Conversation

@wsxiaolin

Copy link
Copy Markdown
Member

Motivation

  • The settings page could be partially inaccessible on short or mobile viewports because the outer wrapper did not act as a scroll container and the app globally hides overflow.
  • Enable vertical touch panning and momentum scrolling so users can reach all settings on mobile and rotated screens.

Description

  • Update src/views/Settings.vue CSS to make .settings-wrapper a scroll container by replacing min-height: 100vh with height: 100dvh and adding overflow-y: auto.
  • Add box-sizing: border-box to avoid padding increasing the visual height and add touch-action: pan-y and -webkit-overflow-scrolling: touch to enable mobile touch scrolling and momentum.
  • No JavaScript or functional behavior changes were made; layout adjustments are scoped to the settings view.

Testing

  • Ran npm run build which completed successfully.
  • Installed Playwright and OS deps with npx playwright install chromium and npx playwright install-deps chromium which succeeded.
  • Executed a Playwright script against /#/s that checked .settings-wrapper styles and attempted programmatic scrolling; the wrapper reported overflowY: "auto" and touchAction: "pan-y", and a small-viewport run showed scrollTop changed from 0 to 215, confirming scrollability.

Codex Task

@wsxiaolin wsxiaolin merged commit cd77b6d into main Jun 7, 2026
4 checks passed
@wsxiaolin wsxiaolin deleted the codex-adoten branch June 7, 2026 05:23
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