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

When in the asset pipeline, less-rails should generate stylesheets as .less templates #8

Closed
torrancew opened this issue Oct 17, 2011 · 7 comments

Comments

@torrancew
Copy link

It would be nice if less-rails generated template stylesheets as .less files. I've forked and pushed an initial attempt to this at the 'with_generator' branch of my fork.

@joliss
Copy link

joliss commented Nov 3, 2011

For everyone's reference, here's the diff: torrancew/less-rails@master...with_generator

(Perhaps you should send a pull request, @torrancew?)

cc @metaskills

@joliss
Copy link

joliss commented Nov 3, 2011

Hm, I wonder if there is a way to get generators for LESS, but still allowing people who only want the less-rails-bootstrap gem to keep using the standard SCSS generators.

@torrancew
Copy link
Author

Good point. I'll investigate this further this weekend.

@metaskills
Copy link
Owner

I started reading deep into the generator code last night and think a few things can be done. I like how others namespace theirs and I am for this vs making all things change (if you have less) installed. So stuff like less:css, etc.

@torrancew
Copy link
Author

Sounds interesting metaskills - I'm fairly new to Ruby and Rails. Any chance I could pick your brain on this a bit some time?

@metaskills
Copy link
Owner

Well, all I did was just looked at the output of $ rails generate -h in the console and queued off of the asset ones. I then started digging deep into the railties source and started reading the code of everything in the generator directory while looking at what other rails gems have written. I'll put some more time into this over the weekend.

@metaskills
Copy link
Owner

So that commit should solve the generator issue. If anyone can test it out and let me know if I missed something before the next release?

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

3 participants