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.

Sign up for GitHub

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

Jump to bottom

Blazor WebView iOS Zoom Support #4996

Closed
TanayParikh opened this issue Mar 2, 2022 · 2 comments
Closed

Blazor WebView iOS Zoom Support #4996

TanayParikh opened this issue Mar 2, 2022 · 2 comments
Assignees
@jfversluis
Labels
area-blazor Blazor Hybrid / Desktop, BlazorWebView platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
Triage

Comments

@TanayParikh
Copy link
Contributor

TanayParikh commented Mar 2, 2022 •
edited

Description

Not sure if this behavior is intentional, but should pinch to zoom be enabled by default like this? Makes the app feel "less native".

Start at 9 seconds:

Simulator.Screen.Recording.-.iPad.mini.6th.generation.-.2022-03-01.at.16.59.41.mp4

Steps to Reproduce

dotnet new maui-blazor
Open on iPad and pinch to zoom.

Version with bug

Preview 13 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iOS

Did you find any workaround?

No response

Relevant log output

No response
@TanayParikh TanayParikh added t/bug Something isn't working area-blazor Blazor Hybrid / Desktop, BlazorWebView labels Mar 2, 2022
@jsuarezruiz jsuarezruiz added this to New in Triage Mar 2, 2022
@jfversluis
Copy link
Member

Thanks for the report @TanayParikh ! It's definitely an interesting one, I ran into that as well. You can easily fix it in HTML by adding this to your <head></head> tags: <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no" />

Now the interesting part is... Is this something we should include in the template by default? Should we arrange this on a .NET MAUI level?

Personally I feel at the very most we should add it to the template so this doesn't happen out of the box but I would say this is the responsibility of the developer working with this. Maybe some people think this is desired behavior. I will take it up with the team for discussion.

@jfversluis jfversluis self-assigned this Mar 2, 2022
@jfversluis jfversluis mentioned this issue Mar 2, 2022
Closed
@jfversluis jfversluis moved this from New to Done in Triage Mar 2, 2022
@jfversluis jfversluis added s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage labels Mar 2, 2022
@jfversluis
Copy link
Member

This is going to be fixed by #4922 exactly in the way I intended to do it :) closing this for now!

@ghost ghost locked as resolved and limited conversation to collaborators Apr 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jfversluis jfversluis

Labels
area-blazor Blazor Hybrid / Desktop, BlazorWebView platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
No open projects
Triage
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Disable BlazorWebView zoom behavior by default dotnet/maui
3 participants
@jfversluis @samhouts @TanayParikh