Skip to content
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

Auto zoom when using Nextcloud on Safari iOS #47328

Closed
juliusknorr opened this issue Aug 19, 2024 · 5 comments · Fixed by #47371
Closed

Auto zoom when using Nextcloud on Safari iOS #47328

juliusknorr opened this issue Aug 19, 2024 · 5 comments · Fixed by #47371
Labels
3. to review Waiting for reviews design Design, UI, UX, etc. papercut Annoying recurring issue with possibly simple fix.

Comments

@juliusknorr
Copy link
Member

I used our web interface a couple of times now on mobile safari and if feels quite strange and is partially also annoying that Safari is automatically zooming to input elements and also contenteditables (e.g. in text) when you focus them.

This used to work differently and in my perspective also should for a web app that is not a pure content page.

This was introduced in ac736b2 as a accessibility fix where I still agree that manually zooming should be possible, however maybe we can get rid of the automatic zoomin.

I found https://stackoverflow.com/questions/2989263/disable-auto-zoom-in-input-text-tag-safari-on-iphone as a possible option.

@nextcloud/designers What do you think about that? Any arguments against changing this?

@juliusknorr juliusknorr added design Design, UI, UX, etc. papercut Annoying recurring issue with possibly simple fix. labels Aug 19, 2024
@susnux
Copy link
Contributor

susnux commented Aug 20, 2024

Tested a bit, all the meta solutions will break android.
So either:

  1. Add that meta attribute only on ios (by user agent)
  2. Increase our font size to 16px (Safari then does not auto zoom)
  3. No zooming again (probably not good)

@jancborchardt
Copy link
Member

@susnux solution 1 and 3 are the same though, no? Meaning blocking zoom on Safari, which is an issue for accessibility.

cc @marcoambrosini regarding the font size question, although we never were at 16px so it must always have happened.

@susnux
Copy link
Contributor

susnux commented Aug 20, 2024

@jancborchardt 1 means we keep the zooming behavior but set maximum-scale=1 when UA is Safari.
This will stop auto-zoom on iOS but keep user zoom. But only works on Safari, on Chrome it breaks user zooming.

BTW 16px is the default font size of (every?) browser, benefit would be that the line height would be 24px instead of 22.5px (which looks sometimes odd due to the fraction).

@jancborchardt
Copy link
Member

@susnux true, we just keep user-scalable. Then solution 1 sounds best and simplest, right? Seems fine to me 👍

@susnux susnux added the 3. to review Waiting for reviews label Aug 21, 2024
@juliusknorr
Copy link
Member Author

Thanks so much for checking that out right away @susnux ❤️

Using the web UI is so much smoother now on iOS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews design Design, UI, UX, etc. papercut Annoying recurring issue with possibly simple fix.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants