Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

stacked detail modal fixes #2339

Merged
merged 1 commit into from
Oct 18, 2024
Merged

stacked detail modal fixes #2339

merged 1 commit into from
Oct 18, 2024

Conversation

manV
Copy link
Member

@manV manV commented Oct 18, 2024

Changes proposed in this pull request:

  • Radix modals don't work well when state is managed globally, so fixed by mounting modal from within modals.

@manV manV requested a review from ramanan-ravi October 18, 2024 07:12
@manV manV merged commit f13842d into main Oct 18, 2024
3 of 5 checks passed
@manV manV deleted the stacked-modal-fixes branch October 18, 2024 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants