Skip to content

Comments

Fix issues with shell and pushing modals early#13990

Closed
PureWeen wants to merge 2 commits intomainfrom
fix_modal_shell_appearing
Closed

Fix issues with shell and pushing modals early#13990
PureWeen wants to merge 2 commits intomainfrom
fix_modal_shell_appearing

Conversation

@PureWeen
Copy link
Member

@PureWeen PureWeen commented Mar 16, 2023

Description of Change

This PR is a continuation of #13025. A few issues were identified with pushing modals to shell before you've attached it to a window.

Fixes #8157

@PureWeen PureWeen requested a review from rmarinho March 16, 2023 16:00
@github-actions
Copy link
Contributor

Thank you for your pull request. We are auto-formatting your source code to follow our code guidelines.

@Eilon Eilon added the area-controls-shell Shell Navigation, Routes, Tabs, Flyout label Mar 16, 2023
@PureWeen PureWeen marked this pull request as draft March 16, 2023 23:36
@PureWeen PureWeen closed this Jun 15, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-controls-shell Shell Navigation, Routes, Tabs, Flyout

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Modal Navigation does not work or throws exception prior to Window Creation/Parenting

3 participants