Attached patch simply removes some unnecessary parameters to the drupal_add_css call.

The behaviour before the patch is to add the codefilter.css file separately and causes an extra HTTP request for no reason. The new behaviour is to allow the CSS file to be aggregated with the others.

CommentFileSizeAuthor
codefilter.css_preprocess.patch675 bytespsynaptic

Comments

psynaptic’s picture

psynaptic commented
Title: Preprocess CSS file » Allow aggregation of CSS file

Better title ;)

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

+1. This seems like a simple one to commit to get some performance benefit.

drurian’s picture

drurian commented

Please get it committed. I have like a hundred modules and codefilter.css is the only one not being aggregated.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Reviewed & tested by the community

indeed.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Reviewed & tested by the community » Fixed

Applied to 6.x and 7.x branches.

Status: Fixed » Closed (fixed)

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