Lots of ubercart core modules are adding their css files on every page load.
It would be more efficient to only add them when they are needed.

Comments

rszrama’s picture

rszrama commented
Status: Active » Postponed (maintainer needs more info)

I'd consider a patch for that, otherwise this has already been improved on in the D6 branch.

NewZeal’s picture

NewZeal commented

I had a lot of problems with ubercart css interfering with wysiwyg and completely crashing the theme in some forms (css not working at all) in Internet Explorer. I went through and removed drupal_add_css from hook_menu in five modules:
uc_attributes
uc_product
uc_roles
uc_order
uc_store
uc_reports
(placing it in various functions in order to hopefully have it in place when needed)

The result is that wysiwyg displays correctly and I have my theme back.

tr’s picture

tr commented
Issue tags: +Ubercart theme layer

Tagging

rickmanelius’s picture

rickmanelius commented

subscribing.

I'm optimizing the site now and I noticed uc_product, uc_store, etc are on every page load. Not a huge deal, but it would be nice if it only added when necessary...

rickmanelius’s picture

rickmanelius commented

Actually, it appears other modules like the date module do this be design for CSS aggregation.

http://drupal.org/node/369159

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Postponed (maintainer needs more info) » Closed (works as designed)