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

[Popover] Falling offscreen, doesn't reposition without window.resize event #32704

Closed
2 tasks done
ntbosscher opened this issue May 9, 2022 · 6 comments
Closed
2 tasks done
Assignees
Labels
component: Popover The React component. support: question Community support but can be turned into an improvement

Comments

@ntbosscher
Copy link

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Current behavior 😯

Popover is partially offscreen until a window.resize event is triggered.

Expected behavior 🤔

Should automatically stay within the window bounds.

Steps to reproduce 🕹

https://codesandbox.io/s/modern-grass-x8043b

Context 🔦

I'm trying to build a user info menu on an AppBar header

Your environment 🌎

`npx @mui/envinfo`
Running on Mac: Chrome Version 101.0.4951.54 (Official Build) (x86_64)

  Output from `npx @mui/envinfo` goes here.
  System:
    OS: macOS 12.3.1
  Binaries:
    Node: 16.14.0 - ~/.nvm/versions/node/v16.14.0/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 8.3.1 - ~/.nvm/versions/node/v16.14.0/bin/npm
  Browsers:
    Chrome: 101.0.4951.54
    Edge: 95.0.1020.44
    Firefox: 99.0.1
    Safari: 15.4
  npmPackages:
    @emotion/react: ^11.9.0 => 11.9.0 
    @emotion/styled: ^11.8.1 => 11.8.1 
    @mui/base:  5.0.0-alpha.79 
    @mui/icons-material: ^5.6.2 => 5.6.2 
    @mui/material: ^5.6.4 => 5.6.4 
    @mui/private-theming:  5.6.2 
    @mui/styled-engine:  5.6.1 
    @mui/system:  5.6.4 
    @mui/types:  7.1.3 
    @mui/utils:  5.6.1 
    @types/react: ^18.0.0 => 18.0.8 
    react: ^18.1.0 => 18.1.0 
    react-dom: ^18.1.0 => 18.1.0 
    typescript: ^4.4.2 => 4.6.4 
@ntbosscher ntbosscher added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label May 9, 2022
@ntbosscher
Copy link
Author

May-09-2022 14-55-33

@ntbosscher
Copy link
Author

Possibly related to: #10595
The difference is that I would expect the Popover to handle the repositioning on it's own when the rendering isn't deferred/updated after the initial open.

@danilo-leal danilo-leal added the component: Popover The React component. label May 10, 2022
@siriwatknp siriwatknp added new feature New feature or request and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels May 31, 2022
@siriwatknp
Copy link
Member

It works correctly in my machine.

Screen.Recording.2565-05-31.at.12.16.03.mov

@siriwatknp siriwatknp added support: question Community support but can be turned into an improvement and removed new feature New feature or request labels May 31, 2022
@ntbosscher
Copy link
Author

ntbosscher commented May 31, 2022 via email

@siriwatknp
Copy link
Member

@ntbosscher I believe it is (Version 102.0.5005.61 (Official Build) (arm64))

@ntbosscher
Copy link
Author

Thank you @siriwatknp! Your response got me digging a bit further.

Looks like the problem is caused by a Font Fingerprint extension I had turned on.
image

Definitely out of the scope of this project. Thank you all for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: Popover The React component. support: question Community support but can be turned into an improvement
Projects
None yet
Development

No branches or pull requests

3 participants