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

iOS Outlook - app bottom navigation overlaps view port #2222

Closed
onmomo opened this issue Mar 14, 2024 · 9 comments
Closed

iOS Outlook - app bottom navigation overlaps view port #2222

onmomo opened this issue Mar 14, 2024 · 9 comments

Comments

@onmomo
Copy link

onmomo commented Mar 14, 2024

We experienced the following issue with the iOS Outlook app.

iOS Outlook app

IMG_4F5E8EFB5BD0-1

The apply (ANWENDEN) button is positioned so that it is always visible at the end of the user’s view. This means that if there is a bottom navigation/toolbar visible it should be shown above it.
CSS implementation details:
The component becomes fill screen by applying height: 100%
It consists of three divs - one for the header, one for the content and one for the footer with the buttons ()
The content div applies flex-grow: 1 to keep the content positioned at the top below the header.

We can't reproduce this with the Teams iOS mobile app, where the mobile app bottom navigation is excluded from the 100% viewport, which is what we expect.

iOS Teams app

image

The same issue can be seen on other parts, that rely on the iOS Outlook bottom navigation bar being excluded from the visible viewport.

iOS Outlook app

IMG_65DCBEE81CD8-1

Please advise, this blocks our teams app store validation.

@ChetanSharma-msft
Copy link

Hello @onmomo - Could you please share the repro steps or let us know if we can test any existing 1st party app to repro your issue?

@ErythroME
Copy link

ErythroME commented Mar 15, 2024

Hi @onmomo, you can fix the height issue in Outlook iOS by using CSS 100dvh (with fallback for compatibility https://caniuse.com/mdn-api_css_dvh_static) as body height.

@onmomo
Copy link
Author

onmomo commented Mar 15, 2024

Hi @onmomo, you can fix the height issue in Outlook iOS by using CSS 100dvh (with fallback for compatibility https://caniuse.com/mdn-api_css_dvh_static) as body height.

Thanks a lot @ErythroME. We will definitely look into this. Looks promising (https://blog.logrocket.com/improving-mobile-design-latest-css-viewport-units/#dynamic-viewport-units). We're not sure yet if vuetify does support it, though.

@sakov-ms
Copy link
Contributor

Thank you @onmomo for raising this issue. Do try out the solution proposed by @ErythroME and let us know if that works.

@onmomo
Copy link
Author

onmomo commented Mar 18, 2024

@ErythroME @sakov-ms
We verified the suggestion internally and found that it won't resolve the reported issue. We leverage vuetifyjs that should deal with mobile webviews, correctly
Please find our tests with chrome and edge mobile.

Chrome Mobile

Screenshot_20240318_171144_Chrome

Edge Mobile

Screenshot_20240318_172323_Edge

Since our implementation also works flawlessly in MS Teams mobile app, this still looks like an issue on MS Outlook mobile app. We cannot even implement a workaround that would look good for MS Teams mobile and MS Outlook mobile since later does somehow prevent to detect the correct viewport height.
Please check this internally.

@sakov-ms
Copy link
Contributor

Thank you for this detailed investigation. We are checking this issue and will provide updates soon.

@AE-MS
Copy link
Contributor

AE-MS commented May 9, 2024

Here's what the Outlook team said about this issue:

The implementation works differently for MS Teams and Outlook for iOS because by design there are differences in the
bottom navigation. Teams's bottom navigation bar does not cover & blur the underlying website, but Outlook for iOS does. Therefore using dvh is the preferred implementation for current support; changing the behavior on Outlook for iOS will also cause regression on other Apps therefore currently there is no plan to change the behavior on Outlook. Using CSS 100dvh (with fallback for compatibility https://caniuse.com/mdn-api_css_dvh_static) as body height should be able to workaround this issue, if it doesn’t work please post example CSS code using the dvh property for us to investigate.

@TrevorJoelHarris
Copy link
Contributor

I'm going to close this due to inactivity from the original reporter. @onmomo if you still have questions or problems feel free to re-open this or open a new issue and we will do our best to help!

Copy link
Contributor

Tell us about your experience!

Hi onmomo! We have closed this issue in our system, but we would love to hear your feedback. To let us know how we did, simply use one of the following emoji icons: 👍 for resolved, 👎 for unresolved, or ❤️ for a great experience on this message. Please use the link for more detailed feedback.

Best regards,
Teams Platform

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants