css optimization skipped for single module builds

[daltonjc]

When using r.js to optimize a single module using the name and out options, require-css doesn't do css optimization.

The compress(css) function in css-builder.js checks for optimizeCss and skips optimization if that's set to 'none'. However, r.js will overwrite optimizeCss with 'none' if an out option was specified with no cssIn option, with the reasoning that if you're doing a single module and it's not a css file then do no css optimization. That logic is great for normal r.js behavior, but for require-css it's likely that single js module you're optimizing contains some css you'd like to optimize as well.

Perhaps we need a different option to check instead of the existing r.js optimizeCss option since r.js does some strange logic with that one? And if that's the case, how is that married with the existing check on optimizeCss since presumably people are relying on that option being checked?

[daltonjc]

I can contribute a fix for this bug if that would help. Though I wouldn't mind an opinion first from a project veteran on whether to just stop respecting the optimizeCss check (feels like a BC break to me), or keep respecting that but then add a new option to override that (feels bloaty). Any other way to fix while leaving optimizeCss?

[guybedford]

A fix for this would be welcome. Perhaps let's just always optimize the CSS if possible?

[daltonjc]

Ok, can do, thanks @guybedford. Think I should add a new option to take its place (that r.js won't change), so that if anyone does get burned by it we can just say "ah, just change it to this option and you're good to go"? Not sure if anyone's depending on non-optimized css or not.

I'll look to take care of this next week.

[guybedford]

Sure, if you think it's important.

[guybedford]

Thanks for looking into this!

[darrenng-dpodium]

Hi @guybedford I am facing this exact issue. Will the merge happen soon?

[dingus9]

Is there a workaround for this? I am being bit by this issue

[Txabs]

Me as well

[alundiak]

I've recently joined to this module maintainer, and I reviewed this issue and dedicated PR. I kinda have some doubts, but still, I will research the problem deeper these days.

@darrenng-dpodium , @dingus9 , @Txabs if you have some new ideas in this area, keep us posted.