- Issue created by @andre.bonon
- @andrebonon opened merge request.
- π§π·Brazil andre.bonon
Attached the patch from MR to use in composer.
- Status changed to Needs review
over 1 year ago 9:59pm 15 August 2023 - π¨π¦Canada joseph.olstad
@andre.bonon,
It might be safer (avoid unexpected results) to use the event.target element as an example:$(event.target).parent().find('.ui-widget-overlay').remove();
or
$(event.target).parent().parent().find('.ui-widget-overlay').remove();
?
I haven't tested this but when I get time I'll look a bit closer at this issue, it likely also affects 2.0.x
- π§π·Brazil andre.bonon
Ok, I've added your suggestion and fixed the other selector as well.
Also, I unbind the event so it doesn't bind the event twice. (That was happening on the /admin/content page) -
joseph.olstad β
committed 16eaa830 on 3.0.x authored by
andre.bonon β
Issue #3381313 by andre.bonon: ConfirmationDialog doesn't remove the...
-
joseph.olstad β
committed 16eaa830 on 3.0.x authored by
andre.bonon β
- Status changed to Fixed
about 1 year ago 7:58pm 22 August 2023 - π¨π¦Canada joseph.olstad
this is fixed in the dev branch, it'll go out in the next tagged release, whenever that is
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
about 1 year ago 6:37pm 27 September 2023 - πΊπΈUnited States slbrassard
This issue is also happening on the 2.x branch of the module. Specifically, I am using version 2.0.28. Patch #7 does apply to that version and fix the issue.
Can you please make the fix on the 2.x branch too? @josepholstad Thanks!
- Status changed to Active
about 1 year ago 8:48pm 27 September 2023 - Status changed to Fixed
about 1 year ago 9:39pm 27 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.