With the added permissions from #738488: Missing permission hook, it is time to consider not aggregating the CSS.

I think most people who use Block Edit are administrators or content editors. Most visitors to our websites don't fall into these categories and therefore don't have access to Block Edit.

Because of this, it doesn't make since to have the thousands of visitors to my website load CSS they will never use. It is a waste of resources and a kill on performance.

Comments

philbar’s picture

philbar commented

Admin Menu is an examples of a module that doesn't aggregate its css.

#187198: Aggregate and compress CSS files not works ?
#671760: admin_menu.css not aggregated - is there a compelling reason?

joelstein’s picture

joelstein commented
Status: Active » Fixed

Though it is possible to grant anonymous users the "view block edit links" and "view node edit links" permissions, I agree that in most cases this module is being used only by administrators or content editors.

Fixed in 6.x-1.x:
http://drupal.org/commitlog/commit/7690/b4eab066d573051dfab182b9167208ed...

A new release should be available shortly.

Status: Fixed » Closed (fixed)

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

maximpodorov’s picture

maximpodorov commented
Status: Closed (fixed) » Active

The module doesn't check user permissions before adding CSS and JS.

joelstein’s picture

joelstein commented
Status: Active » Closed (fixed)

Sorry, but that's a new issue. Feel free to start another ticket if you want.