Remove deprecated Jquery UI library in favor of https://www.drupal.org/node/3067969
to resolve error : Uncaught TypeError: this.$content.resizable is not a function

CommentFileSizeAuthor
#2 3420362-2.patch487 bytessneha_surve

Comments

sneha_surve created an issue. See original summary.

sneha_surve’s picture

sneha_surve commented
StatusFileSize
new 3420362-2.patch487 bytes

Patch fixes modal issue for Drupal 10

james.williams’s picture

james.williams
Location Coventry, UK
commented

This assumes the jQuery UI module(s) are installed; I don't think this project currently requires them, or brings them in as dependencies at all. So these library definitions would either need to become dynamic, so that they're only available when those dependencies are in place, or enforce that those dependencies are installed (which would be even if these libraries aren't used).

sneha_surve’s picture

sneha_surve commented
Status: Needs work » Needs review
james.williams’s picture

james.williams
Location Coventry, UK
commented
Status: Needs review » Needs work

I have reviewed; and the answer is that more work is needed.

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented

Thanks for the patch.
I agree with @james.williams.

Needs at least to add dependencies (see this issue as an example) or replace functionality with core functionality (see this issue).

amanp’s picture

amanp commented

An update for this was included in 3.31.0. See #3427365.

There's an open MR-42 to handle ensuring that the dependencies are enabled.

joseph.olstad’s picture

joseph.olstad
Primary language French
commented
Status: Needs work » Closed (duplicate)
Parent issue: » #3427365: Remove jQuery UI Library's Core dependency and Add module dependency

This is fixed since quite a few releases back

#3427365: Remove jQuery UI Library's Core dependency and Add module dependency