[Popover] Repositions After Pinch Zoom Safari Mobile

[kmsarsam]

When selecting an item from a MenuItem, if you try to zoom in it will go out of sight and when you zoom back in, the list will be repositioned away from its original position

• The issue is present in the latest release.
• I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

We have MenuItems for selecting from a dropdown inside of TextFields. On safari on mobile devices, if you zoom in on the MenuItems, you will no longer be able to see them until you zoom out. Once zoomed out, they are no longer in place with where they were.

Expected Behavior 🤔

When selecting an item, you should be able to zoom in on a mobile device to read the text and be able to select an item then zoom out and the MenuItems should be in the same place.

Steps to Reproduce 🕹

(https://material-ui.com/components/menus/)
This behavior happens on the material-ui site for menus.

Steps:

1. Open the material-ui website on an iPhone or xCode iPhone simulator
2. Open one of the menus
3. Zoom all the way in, then move around while zoomed in
4. Zoom out and the menu will be in a different location out of view

Context 🔦

We have users who would like to navigate through forms zoomed in or would like to zoom in to read the text on the MenuItems without losing their place.

Your Environment 🌎

   System:
    OS: Windows 10 10.0.18363
  Binaries:
    Node: 12.18.4 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\ec\src\Aspen.Canopy.eCommerce\Client\node_modules\.bin\yarn.CMD
    npm: 6.14.6 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 86.0.4240.198
    Edge: Spartan (44.18362.449.0)
  npmPackages:
    @material-ui/core: ^4.5.0 => 4.11.2
    @material-ui/icons: ^4.0.1 => 4.11.2
    @material-ui/lab: ^4.0.0-alpha.28 => 4.0.0-alpha.57
    @material-ui/pickers: ^3.2.4 => 3.2.10
    @material-ui/styles:  4.11.2
    @material-ui/system:  4.11.2
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react: 16.8.2 => 16.8.2
    react: ^16.8.1 => 16.14.0
    react-dom: ^16.8.1 => 16.14.0
    typescript: ^3.5.1 => 3.9.7

[oliviertassinari]

@kmsarsam Which version of iOS, Safari are you using? Do you have a visual illustration?

[kmsarsam]

My iPhone simulator says software version 14.2

…

On Wed, Dec 9, 2020, 3:59 PM Olivier Tassinari ***@***.***> wrote: @kmsarsam <https://github.com/kmsarsam> Which version of iOS, Safari are you using? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#23919 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIZSETPXZ7M4T4PDQUJS5XTST76MLANCNFSM4UUE25MQ> .

[oliviertassinari]

I could reproduce, it seems a bit related to #5750

[oliviertassinari]

It's an issue with the Popover position logic. The Popper component isn't impacted. This issue was already reported in #9802. I think that we need to move forward with the plan of #17636. To have Popover combine Modal + Popper instead of having its own position logic (dropping it).