Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LESS compiler integration #7

Closed
paales opened this issue Nov 11, 2012 · 3 comments
Closed

LESS compiler integration #7

paales opened this issue Nov 11, 2012 · 3 comments

Comments

@paales
Copy link
Contributor

paales commented Nov 11, 2012

Hi There,

I was working on a bootstrap theme and wanted to use the .less files from bootstrap so I can extend it to build additional components. After a quick search I noticed that there was a module available (https://github.com/r8/magento-lesscss), but I didn't really like the implementation (puts the CSS files at different locations and image URL's don't work properly).

Since I am already using your module and think its great, I made a simple fork and added the ability to pre-process the files (https://github.com/ho-nl/Magento-Minify/tree/less).

Based on the file extension a file is preprocessed (future functionality like CoffeeScript or SASS easily implemented:
ho-nl-fork@b963904#diff-0

For development purposes there is less.js available which parses the .less files in realtime.

If you like it, I can make a pullrequest so it can be integrated in the main branch.

@paales
Copy link
Contributor Author

paales commented Nov 14, 2012

Not to spam you or something, but what do you think? :)

@azurams
Copy link
Owner

azurams commented Nov 16, 2012

Hi,
thanks for using my module and for your suggestions, enabling the ability to work with Less files is very good idea and practice.

On my current work project I use Less but I don't use Magento anymore (for the moment, but soon I hope) , I don't have much time to test it but of course, you are welcomed to make a pullrequest and I'll merge your commits in the main branch.

@paales
Copy link
Contributor Author

paales commented Nov 16, 2012

See #8

@paales paales closed this as completed Nov 16, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants