Skip to content

site: fix pricing section horizontal overflow on mobile - #258

Merged
chaitanyagiri merged 1 commit into
mainfrom
site/pricing-mobile-responsive
Aug 23, 2026
Merged

chaitanyagiri merged 1 commit into
mainfrom
site/pricing-mobile-responsive

Conversation

@chaitanyagiri

Copy link
Copy Markdown
Collaborator

Founder report with iPhone screenshots: the pricing section forced the whole page to scroll sideways on mobile Safari.

Cause: Safari sizes elements to their longest option text (the sandbox tier labels are long), and WebKit range inputs carry an intrinsic min-width — the plan grid could not shrink below content width. Fix: min-width: 0 down the flex/grid chain (.pgroup, .plan, .cfg, .cfg select, .seat-ctl range) and width: 100% + text-overflow: ellipsis on the configurator selects. No markup or logic changes; desktop layout unchanged.

Safari sizes <select> elements to their longest option and WebKit range
inputs carry an intrinsic min-width, so the plan cards forced the whole
page to scroll sideways on phones (founder report with screenshots).
min-width:0 down the flex/grid chain (pgroup, plan, cfg, select,
seat slider) and width:100% + ellipsis on the configurator selects.
@github-actions

Copy link
Copy Markdown
Contributor

🚫 This PR is missing its before/after evidence

Every pull request here has to show its work. Screenshots or a short screen recording, before the change and after it.

  • Before — no image or video under that heading
  • After — no image or video under that heading

How to fix it: edit the description, keep the ### Before and ### After headings from the template, and drag an image or video under each. GitHub uploads it inline. This check re-runs the moment you save.

A bug fix with no visible surface still needs it: show the failing behaviour, then the same steps passing. A terminal recording is fine.

Genuinely nothing to show — a CI tweak, a typo, a dependency bump? A maintainer can apply the no-visual-change label. Please don't ask unless it truly has no observable effect.

📖 CONTRIBUTING.md → Evidence is mandatory

@chaitanyagiri
chaitanyagiri merged commit 99726db into main Aug 23, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant