- Issue created by @Grimreaper
- Merge request !9406Issue #3471977 by grimreaper: Drupal.dialog openDialog should use event settings β (Closed) created by Grimreaper
- Issue was unassigned.
- Status changed to Needs review
4 months ago 7:24am 4 September 2024 - Status changed to RTBC
4 months ago 12:48pm 4 September 2024 - πΊπΈUnited States smustgrave
Change seems small enough that doesn't need test coverage I would say.
- Status changed to Fixed
4 months ago 1:02pm 6 September 2024 - π«π·France nod_ Lille
Double checked it turns out the altered settings is working in the current code, this makes it more explicit and remove any ambiguity as to the intent. I think this was a problem when using CustomEvent instead of our own event class, in any case not a bug, but core is more maintainable.
Committed and pushed d075b6c360 to 11.x and b95fd40e05 to 11.0.x and ff33b01fc4 to 10.4.x and 65a3907d98 to 10.3.x. Thanks!
- π«π·France Grimreaper France π«π·
Thanks for the quick merge.
Could you also credit https://www.drupal.org/u/tocab β who was the one finding the problem please?
Automatically closed - issue fixed for 2 weeks with no activity.