The google CDN hosts the jquery.ui.theme.css file here: https://ajax.googleapis.com/ajax/libs/jqueryui/1.8.7/themes/base/jquery..... If one uses hook_css_alter and switches this type from file to external, seven_css_alter will not switch the type back to file when it changes the data key.

CommentFileSizeAuthor
#1 drupal-seven-1937860-1-set-type.patch1020 bytesmikeytown2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeytown2’s picture

Status: Active » Needs review
FileSize
1020 bytes
mikeytown2’s picture

Title: seven_css_alter does not check for type » seven_css_alter does not set the type

Status: Needs review » Needs work

The last submitted patch, drupal-seven-1937860-1-set-type.patch, failed testing.

mikeytown2’s picture

Status: Needs work » Needs review

#1: drupal-seven-1937860-1-set-type.patch queued for re-testing.

Test went fatal on FileDeleteTest->testInUse()

markhalliwell’s picture

Status: Needs review » Reviewed & tested by the community

+1, reviewed and tested

David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/51755eb

(Looks like this is not an issue in Drupal 8, by the way.)

Status: Fixed » Closed (fixed)

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