In DEBUG mode, LESS should work without further ado about bedrock HOT 7 CLOSED

docs/index.rst says you'll need a less compiler, but I think you shouldn't need that unless you want to compile/minify CSS for deployment.

Btw, just adding the LESS script to the template does not seem to work, I think that's a bug in jingo-minify, it fails to set rel="stylesheet/less" for less files so the script does not pick up those files. Sounds fixable :)

from bedrock.

I mentioned the exact same thing to jsocol, and he pointed out that it's actually type="text/less" that need to be set (I think). Regardless, we probably should make this work. I like the fact that it compiles it server-side so that it's not always having to dynamically parse LESS, but it is annoying to have node as a requirement.

We can include the js compiler and spit out the appropriate markup if the node compiler isn't installed. I'll poke around jingo-minify to see if there's a way to get it to spit out the right rel or type, whichever's needed.

from bedrock.

Thanks!

lesscss.org says: <link rel="stylesheet/less" type="text/css" href="styles.less">

from bedrock.

I also agree that precompiling is neat, but I think that's something optional. We could insert the JS compiler iff the PREPROCESS setting is set to False (and TEMPLATE_DEBUG=True, of course), otherwise let node do its thing.

from bedrock.

FWIW, I upstreamed this issue to jingo-minify: jsocol/jingo-minify#20

from bedrock.

Cool. We had a few more problems doing it server-side, so I would like to see this happen. I can try to get this done next week.

from bedrock.

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=738406

from bedrock.