As per the title, when CSS aggregation is on, menu_minipanels all display in page footer.

This is because the module is using D6 syntax for drupal_add_css, instead of D7

  // Load the module's custom CSS.
  drupal_add_css($path . '/css/menu_minipanels.css', 'module', 'all', TRUE);

vs just 

  // Load the module's custom CSS.
  drupal_add_css($path . '/css/menu_minipanels.css');

See: http://api.drupal.org/api/drupal/includes--common.inc/function/drupal_ad...

The attached patch fixes the issue and allows CSS aggregation.

Thanks again for the useful module,

DT

CommentFileSizeAuthor
#2 menu_minipanels-n1258150-d6.patch514 bytesdamienmckenna
menu_minipanels.fix-css-aggregation.patch710 bytesdavidwhthomas

Comments

digibeetle’s picture

digibeetle commented

Hi, just wanted to say thanks because your solution indeed fixes this nasty problem!!!
I am building a site and am almost done, so i switched on aggegration and... what a nasty sight :-( However searching the issue queue quickly gave me this solution :-)

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new menu_minipanels-n1258150-d6.patch514 bytes

Thanks. I'm going to change the same line for D6 too (see patch) as there's no point in specifying the additional arguments when they are only passing in the normal defaults anyway.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks, this has been committed.

digibeetle’s picture

digibeetle commented

I can confirm RC2 fixes this :-)

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Add api doc link