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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

maximpodorov’s picture

TwoD’s picture

Cool, 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?

maximpodorov’s picture

elpino’s picture

Issue tags: +Worked for me with CSS aggregation enabled.
elpino’s picture

Issue tags: -Worked for me with CSS aggregation enabled.

Worked for me with CSS aggregation enabled.

*oops sorry for the issue tag, clicked on wrong text box

  • TwoD committed 2bedc7b on 7.x-2.x authored by maximpodorov
    - #2496419 by maximpodorov, elpino: Fixed trying to load CSS files...
TwoD’s picture

Status: Needs review » Fixed

Great, thanks for the patch and the review!

Btw, maybe that should be a tag, considering all the problems aggregation used to cause hehe. ;)

Status: Fixed » Closed (fixed)

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