[Popover] Fix missing type export getOffsetTop & getOffsetLeft

[rart]

Wondering if there's a reason why getOffsetTop and getOffsetLeft aren't exported from the index? They're actually exported from the main Popover.js file but not from the index or typings. Almost seems like a miss.

We've found use for these when programmatically supplying a position to the Popover component (i.e. anchorReference: 'anchorPosition' and anchorPostion: { top, left }).

To use them, we have to ts-ignore the line due to the lack of typings and import from the 4th level which is meant to be private.

// @ts-ignore
import { getOffsetLeft, getOffsetTop } from '@mui/material/Popover';

It'd be nice to have them public

import { getOffsetLeft, getOffsetTop } from '@mui/material/Popover';

• I have followed (at least) the PR section of the contributing guide.

[mui-bot]

Details of bundle changes

Generated by 🚫 dangerJS against 2df4bc7

[oliviertassinari]

To be fair, there is a high chance that we will rewrite Popover to use the positioning logic of Popper: #17636. So these methods will likely go deprecated. Though, it's great to have it in the types, so we can add a clear @deprecated mention in the future!

[oliviertassinari]

Wondering if there's a reason why getOffsetTop and getOffsetLeft aren't exported from the index?

AFAIK, the exports were meant for maintainers, so we could write unit tests. I mean, I don't see the use case for developers.

[rart]

I don't see the use case for developers.

@oliviertassinari as briefly explained on the PR description, we found use for them when programmatically calculating the Popover's position (when using anchorReference: 'anchorPosition') instead of supplying the actual element for the Popover to internally figure it out. Not sure the full details are of interest, happy to provide further detail if so, but in essence: the Popover props come from Redux and since having non-serializables on the state is a bad practice, we store the top & left and the getOffset* functions were useful to compute these. They saved us from duplicating pretty much same logic. 🤷🏻‍♂️