Skip to content

Develop Branch PR: Bug-fix: Nested Modals fix#1252

Closed
RyanSawyer6003 wants to merge 2 commits intoitflow-org:developfrom
RyanSawyer6003:develop
Closed

Develop Branch PR: Bug-fix: Nested Modals fix#1252
RyanSawyer6003 wants to merge 2 commits intoitflow-org:developfrom
RyanSawyer6003:develop

Conversation

@RyanSawyer6003
Copy link

This PR fixed nested modal window behavior on close. Each modal now is in it's own container and parent modals are not closed on closing of child modals.

Added fix for nested modals closing parent modals
Previous Commit didn't correct issue with nested modals. This commit changes how modals are created to put each one in it's own unique container so that closing a nested modal does not close the parent container
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello & Welcome! :)

Thanks for taking the time to help improve ITFlow. We're excited to review your contributions - we'll review this PR as soon as we can!

Whilst you're waiting, please feel free to check out the forum.

Just so you know, all contributions to ITFlow are licensed under the GNU GPL. By contributing you grant us a perpetual & irrevocable license to include your work in ITFlow.

@sonarqubecloud
Copy link

@RyanSawyer6003
Copy link
Author

Nevermind... I jumped the gun. This does work on closing nested modals, but not on saving new data. Closing PR will submit new when properly fixed.

@johnnyq
Copy link
Collaborator

johnnyq commented Dec 10, 2025

Your right you would almost have to refresh the parent model once the child modal submits so it can get any new data?

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