fix(events): stop iOS Safari zoom by wrapping long user content (Issue 1131) - #1136
Merged
Conversation
…om (Issue 1131) iOS Safari auto-zooms the viewport when a focused input, textarea, or select computes font-size below 16px, and the viewport meta tag does not disable user-scalable, so the user is left zoomed in with no way to zoom back out without manually pinching. text-sm (14px) was used on the shared TextField, Textarea, and Select components plus a few call sites duplicating the same classes (CommentComposer, LocationField, DateTimePicker time input, EmailBlastDialog textarea). Bumped the base to text-base (16px) with an md:text-sm override to keep the compact desktop look.
…overflow (Issue 1131) The reported symptom — the event page always loads zoomed in on iOS Safari and re-zooms in whenever you submit a form — is horizontal overflow, the same bug class as Issue 611. When a user-entered field holds a long unbroken token (a pasted url, a spaceless location, a zelle handle), it has no break opportunity and pushes the layout wider than the phone viewport; iOS Safari then renders the whole page at a zoomed-in scale and resets to it on re-layout (e.g. after a submit). It is not the input-focus auto-zoom the earlier font-size change targeted. Issue 611 only covered the title + description. This adds break-words [overflow-wrap:anywhere] to the fields it missed: the location link, the "other" link label, the price/zelle cost lines, and comment + reply bodies (the field most likely to overflow right after a submit). Adds regression tests mirroring the 611 test.
leahpeker
changed the title
leahpeker
marked this pull request as ready for review
July 24, 2026 22:10
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes Issue 1131 — on iOS Safari the event detail page always loads zoomed in (not on focus), and re-zooms in whenever you submit a form (e.g. an RSVP or a comment). You have to pinch back out each time.
Root cause (corrected)
This is horizontal overflow, the same bug class as Issue 611 — not the input-focus auto-zoom that the first commit on this branch targeted.
When a user-entered field holds a long unbroken token (a pasted URL, a spaceless location, a zelle handle), it has no break opportunity and pushes the layout wider than the phone viewport. iOS Safari then renders the whole page at a zoomed-in scale, and resets to that scale on any re-layout — which is exactly why it "zooms back in" right after you submit a form and the list re-renders.
Issue 611 added
break-words [overflow-wrap:anywhere]to the title and description only. The sibling fields that render free-form user text were missed:priceandzelle: …whitespace-pre-wrapalone does not break a long token; this is the field most likely to overflow immediately after a submit)This PR adds the same wrap treatment to all of them, with regression tests mirroring the 611 test.
Note on the earlier font-size change
The first commit raised text inputs from 14px → 16px (
text-base+md:text-sm) to stop iOS Safari's focus auto-zoom. That's a real, separate iOS annoyance and is harmless, so it's kept — but it is not what fixes Issue 1131. The overflow fix above is the actual fix.Test plan
make agent-frontend-typecheck— passesbreak-words [overflow-wrap:anywhere](40+ event tests pass)