-
Notifications
You must be signed in to change notification settings - Fork 108
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
Cruds in multiple-layered modals behave strange #1732
Comments
This comment was marked as resolved.
This comment was marked as resolved.
linking here another Modal related issue - #1699 |
This comment was marked as resolved.
This comment was marked as resolved.
@ibelar Any idea how to fix this critical and major bug? Really limiting factor to make use of modals unfortunately. Seems also that loading a Virtual Page as Modal from another Modal shows exactly the same issue as above - so not necessarily linked to Crud only, but loading a Modal from a Modal (and closing it, reopening another dynamically loaded Modal). |
that should be simpler to debug, please post a minimalistic code and exact steps to reproduce |
Hard to reproduce, in some instanced I could make it happen that also a modal was empty. In sample code of PR #1764 one of the modals causes a memory leakage. Here is the memory leakage video - you can also see I have to close the non-loading modal twice to go back which resembles the error in my first video. - #1764 (comment) |
Highly strange - If I add another static modal inside the addModalAction right after $modalvp3 = ... using And the doubling of the modals, happens after I have opened twice the modals and closed with different records. If I then open the second lay modal a third time, the HTML code shows that two new modals are added when opening it, instead of one: |
I confirm the initial submited issue and simplified the repro code to https://github.com/atk4/ui/blob/3fd832f729ad230dd918958c9fac09181266095f/demos/_unit-test/crud_gh1732.php using already present demo models |
Seems a logic error on cascaded cruds/modals.
See video here:
Bildschirmaufnahme.2022-02-07.um.21.40.59.mov
The text was updated successfully, but these errors were encountered: