-
Notifications
You must be signed in to change notification settings - Fork 969
fix(web): prevent iOS auto-zoom on chat input focus #1292
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1093,7 +1093,7 @@ function ComposerPromptEditorInner({ | |
| contentEditable={ | ||
| <ContentEditable | ||
| className={cn( | ||
| "block max-h-[200px] min-h-17.5 w-full overflow-y-auto whitespace-pre-wrap break-words bg-transparent text-[14px] leading-relaxed text-foreground focus:outline-none", | ||
| "block max-h-[200px] min-h-17.5 w-full overflow-y-auto whitespace-pre-wrap break-words bg-transparent text-[16px] sm:text-[14px] leading-relaxed text-foreground focus:outline-none", | ||
| className, | ||
| )} | ||
| data-testid="composer-editor" | ||
|
|
@@ -1104,7 +1104,7 @@ function ComposerPromptEditorInner({ | |
| } | ||
| placeholder={ | ||
| terminalContexts.length > 0 ? null : ( | ||
| <div className="pointer-events-none absolute inset-0 text-[14px] leading-relaxed text-muted-foreground/35"> | ||
| <div className="pointer-events-none absolute inset-0 text-[16px] sm:text-[14px] leading-relaxed text-muted-foreground/35"> | ||
|
Comment on lines
1096
to
+1107
|
||
| {placeholder} | ||
| </div> | ||
| ) | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting
maximum-scale=1.0in the viewport meta tag effectively disables pinch-zoom across the entire app on mobile browsers, which is an accessibility regression (users who rely on zoom can no longer enlarge text/UI). Consider removingmaximum-scaleand addressing iOS input auto-zoom viafont-size >= 16pxon focus/for mobile (or other targeted iOS-only styling) instead of globally restricting zoom.