Dashboard: allow pinch-to-zoom by dropping maximum-scale on dashboard sections - #113337
Merged
Conversation
… sections Scope maximum-scale removal to dashboard sections via a new Head allowZoom prop; other surfaces keep maximum-scale=1.
|
WordPress.com
Automattic for Agencies
|
p-jackson
marked this pull request as ready for review
August 6, 2026 08:30
matticbot
added
the
[Status] Needs Review
fushar
approved these changes
Aug 6, 2026
9 tasks
arthur791004
reviewed
Aug 6, 2026
| branchName={ branchName } | ||
| inlineScriptNonce={ inlineScriptNonce } | ||
| faviconUrl={ headFaviconUrl } | ||
| allowZoom={ isDashboardSection } |
Contributor
There was a problem hiding this comment.
Would it be better to disable zoom on log-in page?
Member
Author
There was a problem hiding this comment.
😮💨 Yes you're probably right. You've highlighted that I'm basically taking a shortcut. Now I have to decide whether the shortcut is warranted 😭
I think the proper fix is that the login page (and basically all pages in Calypso, but there's fewer and fewer of them over time) should be using the input controls from @wordpress/components which mean that the auto zoom doesn't happen.
Contributor
There was a problem hiding this comment.
We can move this forward and address it later 🙂
arthur791004
approved these changes
Aug 7, 2026
github-actions
Bot
removed
the
[Status] Needs Review
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
4 participants
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.
Fixes DOTMSD-1473
Proposed Changes
maximum-scale=1from the viewport meta tag for the dashboard sections (dashboard-dotcom,dashboard-ciab,dashboard-a4a).Headkeepsmaximum-scale=1by default; a newallowZoomprop opts a surface out. Every non-Dashboard surface is unchanged.Why are these changes being made?
maximum-scale=1blocks pinch-to-zoom, which is an accessibility problem for low-vision users.It was re-added in Oct 2025 by "Disable zoom for login page" (#106160), but applied to the shared
Headcomponent, so it affected every Calypso page rather than just login. Scoping the removal to the Dashboard restores zoom where we control the surface (consistent wp-core components) while leaving login's intended behaviour intact.History of
maximum-scale=1in the viewport meta:53bde8c3f9ce73971877a2head/index.jsx468644b51e9e2195a9e504Mobile Safari note: iOS 10+ ignores
maximum-scale/user-scalable=noto always permit pinch-zoom, so this only changes behaviour on Android (which honours it). iOS input focus-zoom is unaffected either way.Testing Instructions
/sites) and confirm pinch-to-zoom works.yarn test-client client/components/head/test/index.jsxpasses.Considerations
Scoped to all three dashboard variants (dotcom/ciab/a4a) since they share the same client.
allowZoomis section-based rather than env-gated, so it applies wherever the dashboard renders.Pre-merge Checklist