This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

Popup: allow to specify mountNode #1177

Closed

layershifter opened this issue Apr 5, 2019 · 1 comment

Labels

vsts

Member

layershifter commented Apr 5, 2019 •

edited

Loading

Feature Request

Problem description

It's not possible to override container in React.createPortal() inside of Popup: https://github.com/stardust-ui/react/blob/v0.27.0/packages/react/src/components/Popup/Popup.tsx#L234

Proposed solution

Introduce a new prop mountNode/context, ???. It should also handle accessibility and targets in EventListener, too.

The text was updated successfully, but these errors were encountered:

pkumarie2011 added the vsts label

Member Author

layershifter commented May 9, 2019

Fixed in #1288.

layershifter closed this as completed

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