[Popper] Use placement viewport instead of window (#22748)

mui · Sep 28, 2020 · 8ea6bbf · 8ea6bbf
1 parent e24cb57
commit 8ea6bbf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/material-ui/src/Popper/Popper.js b/packages/material-ui/src/Popper/Popper.js
@@ -129,7 +129,7 @@ const Popper = React.forwardRef(function Popper(props, ref) {
           : {
               // It's using scrollParent by default, we can use the viewport when using a portal.
               preventOverflow: {
-                boundariesElement: 'window',
+                boundariesElement: 'viewport',
               },
             }),
         ...modifiers,

diff --git a/packages/material-ui/src/Popper/Popper.test.js b/packages/material-ui/src/Popper/Popper.test.js
@@ -264,14 +264,14 @@ describe('<Popper />', () => {
       );
     });
 
-    it('sets preventOverflow to window when disablePortal is false', () => {
+    it('sets preventOverflow to viewport when disablePortal is false', () => {
       const popperRef = React.createRef();
       const { getByRole } = render(<Popper {...defaultProps} popperRef={popperRef} />);
       // renders
       expect(getByRole('tooltip')).to.not.equal(null);
       // correctly sets modifiers
       expect(popperRef.current.options.modifiers.preventOverflow.boundariesElement).to.equal(
-        'window',
+        'viewport',
       );
     });
   });