[Popover] Fix wrong getContentAnchorEl definition (#12562)

* Fixed missing null from getContentAnchorEl. * let's merge
mui · Aug 18, 2018 · 4f3ef62 · 4f3ef62
1 parent 4dabe25
commit 4f3ef62
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/packages/material-ui/src/Popover/Popover.d.ts b/packages/material-ui/src/Popover/Popover.d.ts
@@ -25,7 +25,7 @@ export interface PopoverProps
   anchorReference?: PopoverReference;
   children?: React.ReactNode;
   elevation?: number;
-  getContentAnchorEl?: (element: HTMLElement) => HTMLElement;
+  getContentAnchorEl?: null | ((element: HTMLElement) => HTMLElement);
   marginThreshold?: number;
   modal?: boolean;
   ModalClasses?: ModalClassKey;

diff --git a/packages/material-ui/src/Popover/Popover.js b/packages/material-ui/src/Popover/Popover.js
@@ -239,7 +239,8 @@ class Popover extends React.Component {
           'Material-UI: you can not change the default `anchorOrigin.vertical` value ',
           'when also providing the `getContentAnchorEl` property to the popover component.',
           'Only use one of the two properties.',
-          'Set `getContentAnchorEl` to null or leave `anchorOrigin.vertical` unchanged.',
+          'Set `getContentAnchorEl` to `null | undefined`' +
+            ' or leave `anchorOrigin.vertical` unchanged.',
         ].join('\n'),
       );
     }