1) In drupal core items are ordered differently if they're set to $preprocess or not, even with $preprocess_css disabled.(see drupal_add_css around where it's making it's different style tags and pay look how $preprocess is used). Copied how drupal core does it as far as order; results in 3 style tags where 1 was (1 for module, 1 for theme, 1 for preprocess/other).

2) 960 re-orders css so it's css is at top and puts that into css_alt instead of css (not sure why it doesn't just use $vars[css]). May be an issue for 960. Added a check for css_alt into my patch also since it's a common theme).

Ee, and while investigating I separated the processing code into another function with a variable around it so that people can disable it from running twice when they want to (which they will need to do for 960 since 960 overrides $vars[styles]).

CommentFileSizeAuthor
unlimited_css_960_order.patch3.49 KBhefox
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

donquixote’s picture

Could you have a look at the following isue and test the patch?
#693180-2: Overriding module CSS files does not work with unlimited_css

ademarco’s picture

Status: Active » Needs review

I'm currently testing this patch against donquixote's #693180: Overriding module CSS files does not work with unlimited_css

ademarco’s picture

hefox , donquixote's patch it's committed to HEAD, you may want to test your patch against it.

donquixote’s picture

Status: Needs review » Closed (fixed)

I guess this is fixed in the most recent version.
Please reopen, if it is not.