Skip to content

Ability to style the Popover overlay #5231

Open

Description

@deebov

Provide a general summary of the feature here

Would be very useful to style the Popover overlay the same way as Modal and ModalOverlay.

🤔 Expected Behavior?

You should not be able to interact with outside elements while the popover is open.

😯 Current Behavior

If there are elements with zIndex set, you're able to click them while the popover is still open. That's due to the Popover Overlay not having any zIndex which makes it possible to click other elements with zIndex.

💁 Possible Solution

Either the popover overlay already comes with zIndex or an ability to customize the overlay.

🔦 Context

We have clickable elements with higher zIndex and if you try to close the popover by clicking outside, you can accidentally interact with an element with higher zIndex than the overlay.

💻 Examples

No response

🧢 Your Company/Team

No response

🕷 Tracking Issue

No response

Activity

  1. reidbarber commented on Oct 18, 2023

    @reidbarber
    Member

    Related to discussion in #5126. This would be nice to have, and since it's non-breaking it doesn't need to be added before RAC GA.

  2. Talor-A commented on Dec 1, 2023

    @Talor-A

    hi @reidbarber , this is a niche blocker for us as well. we have an electron app we use the popovers in, and the overlays need this special style rule applied:

     -webkit-app-region: no-drag;

    otherwise, window titlebars eat click events and prevent modals from being closed.

    all we need is some way to target the underlay with styles, either by className, rendering a <Overlay /> element that replaces the built in one, or some other method as a workaround.

  3. devongovett commented on Dec 2, 2023

    @devongovett
    Member

    I think we could add an API similar to <ModalOverlay>, but for popovers. By default, <Popover> would include the overlay, but if you want to customize it, you could wrap it in a <PopoverOverlay>.

  4. deebov commented on Dec 4, 2023

    @deebov
    ContributorAuthor

    @devongovett That would be ideal!

  5. levrik commented on Sep 20, 2024

    @levrik

    This is really needed as otherwise Popovers (Menus in our case) inside a Modal Dialog don't overlay the Modal Dialog which uses z-index as written in code examples from the documentation.

    As a workaround I apply styles to [data-testid="underlay"] for now but TBH this feels hacky.

  6. moved this from ✏️ To Groom to 🩹 To Triage in RSP Component Milestoneson Nov 4, 2024
  7. xamgore commented on Dec 13, 2025

    @xamgore

    That would also be helpful for our own context menu implementations. There are a few things to consider:

    • How to reposition the popover on subsequent right-clicks
    • How to block the browser’s context menu on following right clicks outside the popover
    • How to access underlay without the [data-testid="underlay"] hack

    Context menus are common among desktop users in professional software such as Linear. From an accessibility perspective, couldn't long-presses help here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions