Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
The theme can remove some CSS files from other modules. E.g. Zen removes three CSS files:
stylesheets[all][] = system.menus.css
stylesheets[all][] = system.messages.css
stylesheets[all][] = system.theme.css
WYSIWYG loads all CSS files defined by the theme including these files which does not exist in theme. File existence should be checked before adding CSS files to the editor.
Comment | File | Size | Author |
---|---|---|---|
#1 | wysiwyg-check_css_existence-2496419-1.patch | 523 bytes | maximpodorov |
Comments
Comment #1
maximpodorov CreditAttribution: maximpodorov commentedComment #2
TwoDCool, thanks for the patch! I'll test it together with a bunch of other changes I have planned for dealing with stylesheets later, working on a few other things right now.
Does this work when CSS aggregation is enabled as well?
Comment #3
maximpodorov CreditAttribution: maximpodorov commentedYes, this should work with aggregation. The core performs the same checks:
http://cgit.drupalcode.org/drupal/tree/includes/common.inc?h=7.x#n3613
http://cgit.drupalcode.org/drupal/tree/includes/common.inc?h=7.x#n3485
Comment #4
elpino CreditAttribution: elpino as a volunteer commentedComment #5
elpino CreditAttribution: elpino as a volunteer commentedWorked for me with CSS aggregation enabled.
*oops sorry for the issue tag, clicked on wrong text box
Comment #7
TwoDGreat, thanks for the patch and the review!
Btw, maybe that should be a tag, considering all the problems aggregation used to cause hehe. ;)