-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Accessibility/UX: Why adding maximum-scale=1.0? #15024
Comments
It’s added to prevent accidental zooming. For example native apps can’t be zoomed either (except by built-in 3rd party accessibility apps) so web apps should not be either, just because they live in a browser. |
case closed? @jancborchardt @Volker-E |
@DeepDiver1975 With the input of @jancborchardt it seems that the decision was made earlier. My standpoint is, that people are used to zooming in the browser on the smartphone and the case of accidentally zooming is far more uncommon than the wish to zoom in, in order to have ease of interaction. With the current settings you're not able to zoom in on the image, although that's a very common usecase of image apps. |
Pinch to zoom works on iPads only unfortunately. I'd love it if someone could make it work on more devices. |
@oparoz aren’t there other js zoom/gesture libraries which work on other devices as well? |
@jancborchardt - the problem is that it's the bigshot library which is controlling the gestures and it's apparently not detecting all possible events. I played with the library once, but couldn't figure it out. |
I’m closing this since the general interface should just not be zoomable. However, the Gallery or other viewers should make it possible to zoom images. |
Currently the viewport meta tag is at layout.base.php:
and at the two other layout* files:
What are the reasons for not letting users zoom in on smartphones/tablets? Was it some old, unreliabled iOS behavior?
See dhg/Skeleton#173 for a collection of pro and con arguments in a similar. I'm clearly strongly for the accessibility/less native app approach, allowing users to zoom in.
The text was updated successfully, but these errors were encountered: