fix(Popover): reverted appendTo to document body

[thatblindgeye]

What: Closes #9237

We could instead update Popover's position prop to "auto", which would cause it to reposition before scrolling, though there may still be an issue of it getting cut off by a masthead depending on the layout.

There were also some other components that might be affected (menu like components), but since they don't use a focus trap like Popover does, I'm hesitent to revert those appendTo props to document body. Those could also have scrollbars to reduce the height and avoid the issue, though I don't believe the new dropdown/select currently can be passed the necessary props to achieve that (could be a followup for after v5)

Additional issues:

[patternfly-build]

Preview: https://patternfly-react-pr-9252.surge.sh

A11y report: https://patternfly-react-pr-9252-a11y.surge.sh

[kmcfaul]

The Dropdown/Select should allow any popper property to be passed through popperProps because it is spread directly to Popper. It's a little confusing because those interfaces aren't really functioning as an interface so much as a hook + mask for our docgen. We may want to enforce those interfaces better, and/or expose more props on them.

Everything here lgtm.

[tlabaj]

The Dropdown/Select should allow any popper property to be passed through popperProps because it is spread directly to Popper. It's a little confusing because those interfaces aren't really functioning as an interface so much as a hook + mask for our docgen. We may want to enforce those interfaces better, and/or expose more props on them.

Everything here lgtm.

@kmcfaul can you open up a follow up issue/spike for this.

[tlabaj]

lgtm