I think there is no need to load the modules .css files using the .info files because they are loaded by default.

CommentFileSizeAuthor
redundant-stylesheets-loaded-in-info-files.patch2.46 KBmunzirtaha

Comments

ksenzee’s picture

ksenzee
she/her
commented

I'm not sure I understand. Where else are these .css files being loaded?

bleen’s picture

bleen commented

huh? I dont understand. Can you give an example?

(cross posted with ksenzee ... but we're both confused so thats ok)

munzirtaha’s picture

munzirtaha commented
Status: Needs review » Active

Umm! It's was me actually who got confused and didn't check enough. I found many modules are not using the .info file to load the stylesheet and thought it's loaded by default like the *-rtl.css files. Now, after a second thought I found that some modules e.g

block, color, profile, locale, openid, overlay, profile, simpletest, system, taxonomy, update load their *.css files using drupal_add_css() function,

whereas others like:
aggregator, book, comment, field, forum, node, poll, search, simpletest, user
load the *.css files from the *.info file. Any reason for this inconsistency?

bleen’s picture

bleen commented
Status: Active » Closed (works as designed)

there are reasons ... if you read through #769226: Optimize JS/CSS aggregation for front-end performance and DX you'll see that a lot depends on what css files should end up in the aggregated css file for anon users and whet files shouldn't

munzirtaha’s picture

munzirtaha commented

Thanks for the clarification