Fix: Menu: reconsider the modal behavior

[Rishit30G]

What?

Fixes #68830

Why?

As discussed in the accessibility bug scrub, The modal behavior is unnecessary, it would be better if the menus did not behave as modals.

How?

This PR adds modal = {false} to Menu.Popover to prevent the modal like behaviour in the Page Section

Testing Instructions

• Go the editor
• Go to Pages
• Open the actions dropdown
• Observe that the modal behaviour is absent and one can easily select other items present on the page despite the dropdown being open

Screenshots or screencast

Screen.Recording.2025-01-29.at.8.36.40.AM.mov

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: Rishit30G <rishit30g@git.wordpress.org>
Co-authored-by: joedolson <joedolson@git.wordpress.org>
Co-authored-by: carolinan <poena@git.wordpress.org>
Co-authored-by: afercia <afercia@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[Rishit30G]

I just wanted to check if everything looks good or if there are any improvements we should consider 🙇🏻
cc: @carolinan

[youknowriad]

Thanks @Rishit30G for picking this up after the a11y bug scrub! DataViews now uses the Menu component from @wordpress/ui (#81783), so the Menu.Popover this patches no longer exists. Whether the menu should be modal is still under discussion in #68830 (see mirka's note about scrolling). I'm closing this as obsolete. If you'd like to continue, a fresh PR against the new Menu.Root is very welcome once there's agreement on the issue.