I'm just wondering why, when superfish.css is included with drupal_add_css, the 'preprocess' parameter is set to FALSE? This means superfish.css will not be aggregated with other CSS files. Since the superfish menu is usually replacing the primary links, and is thus visible on every page, I think this parameter should be changed to TRUE or omitted. It works fine on my site and reduces the number of css files served by 25%, from 4 to 3 :)

Comments

Jeff Burnz’s picture

Jeff Burnz commented

I agree, its an oversight on my part.

danepowell’s picture

danepowell commented

:) Glad I could help

danepowell’s picture

danepowell commented
Status: Active » Reviewed & tested by the community
danepowell’s picture

danepowell commented

Anything stopping this from being committed? Just trying to clear out my issue queue here...

Jeff Burnz’s picture

Jeff Burnz commented

Well, not really, except that I am rewriting a new version of the theme so that has kept me busy, also I am traveling at the moment hence my poor rate of reply to the issue queue, I'll be home in a week or so and things can get cleared up. Cheers.

Jeff Burnz’s picture

Jeff Burnz commented
Status: Reviewed & tested by the community » Closed (won't fix)

PR D6 is now end of life product and support will be deprecated, only critical bug fixes will be considered, all development must go to D7.