Would be a nice option to have. Would help to reduce JS aggregate fragmentation & would push more code to the Google CDN.

CommentFileSizeAuthor
#1 1315160-1-advagg_jquery_ui_cdn.patch2.34 KBneilnz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

neilnz’s picture

neilnz CreditAttribution: neilnz commented
Status: Active » Needs review
FileSize
1315160-1-advagg_jquery_ui_cdn.patch2.34 KB

Given that the jquery_ui module no longer adds jquery-ui.js, rather it adds ui.module.min.js etc (and deps), the current advagg_js_cdn module doesn't handle jquery_ui anymore.

Here's a patch that makes it replace all instances of jQuery UI files with the single bundle from Google, making the functionality work again.

Basically it works by no longer searching for a single file, but rather any file added from the jquery_ui module's JS dir.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Fixed

Thanks for the patch it has been committed.
http://drupalcode.org/project/advagg.git/commitdiff/9ed6b6bcc24df2de578d...

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.