- Issue created by @lexfunk
- First commit to issue fork.
- Status changed to Needs review
about 1 year ago 4:28am 15 November 2023 - last update
about 1 year ago Patch Failed to Apply - 🇮🇳India viren18febS
Hi @lexfunk
I have added a patch with all the changes, please review
Thanks - last update
about 1 year ago Patch Failed to Apply - 🇧🇪Belgium andreasderijcke Antwerpen / Gent
+1 for this! So we can lower the amount of dead jQuery UI weight ;)
- 🇩🇪Germany Anybody Porta Westfalica
Could someone explain, if it has simply been forgotten to remove the dependency in the linked issue or is it by design until the core issue is fixed?
There should be details on that in the issue summary, so we can decide if this can be RTBC'd as of #5 or not.
- Status changed to Needs work
12 months ago 9:21am 16 February 2024 - 🇧🇬Bulgaria pfrenssen Sofia
The patch doesn't work. It adds back a dependency on the
core/jquery.ui.dialoglibrary which was removed 2 years ago in 📌 Actually remove deprecated jquery_ui libraries from core Fixed . We should keep usingcore/drupal.dialoginstead.In any case the scope of this issue is to remove these two contrib modules from the composer.json file. These two are not used any more and are safe to remove.
Drupal core is still working on the jQuery UI integration but we can expect this to be provided natively without requiring changes on our end (see 📌 Add a native dialog element to deprecate the jQuery UI dialog Needs work ). If any changes would be needed then we can handle it in a separate issue, and it will not happen before Drupal 11.
- Status changed to Closed: duplicate
12 months ago 9:24am 16 February 2024 - 🇧🇬Bulgaria pfrenssen Sofia
Closing as duplicate of 🐛 Focal point still dependent on jquery_ui_draggable Fixed which has a working patch.