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

Inject the autoprefixer-rails CSS post-processor if present in the load path #68

Closed
wants to merge 1 commit into from

Conversation

nz
Copy link
Contributor

@nz nz commented Dec 23, 2013

Per my recent issue, #67, this is the patch I'm using to add support for the autoprefixer-rails gem in my local fork. It's a really handy gem which handles CSS vendor prefixes automatically, basically replacing most of my use cases for Compass and Bourbon and the like.

My first stab here loads the gem in Environment#initialize if present in the load path. I'm not sure this is the best approach for everyone. Personally, in the day and age of Bundler, the presence of the gem in the load path is enough of an opt-in for me. But that may be too liberal for others.

So I'm open to suggestions on better modularity here. The trick is having access to the right scope for the Sprockets environment object, balanced with some better opt-in from the user to include the autoprefixer.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling ef143b1 on nz:autoprefixer-rails into 9ac861a on ixti:master.

@ixti
Copy link
Contributor

ixti commented Dec 25, 2013

I will add this support but in a bit different way. Hope to get to this question later this week.

@farneman
Copy link

farneman commented Mar 8, 2014

Any update on when this might get included?

@marbiano
Copy link

@ixti Are you still planning to add support in your own way for this?

@ixti
Copy link
Contributor

ixti commented Apr 24, 2014

I plan, but my schedule is always interrupted. Pretty busy these days, but hope will have some time to close outstanding bits and release shiny new version with lots of improvements and features ;))

@ixti
Copy link
Contributor

ixti commented May 25, 2014

Merged in as bd898d4

@ixti ixti closed this May 25, 2014
@ixti
Copy link
Contributor

ixti commented May 25, 2014

Thanks!

@silvenon
Copy link
Contributor

So all I need to do is add autoprefixer-rails to my Gemfile? How do I configure Autoprefixer?

@lifeiscontent
Copy link

@silvenon did you figure this out? I am wondering the same thing.

@ixti
Copy link
Contributor

ixti commented Sep 9, 2014

There's a pull request waiting to be merged that will allow this. Had no
time over a weekend but will jump on that pr as soon as possible, hopefully
this week.
On 9 Sep 2014 01:26, "Aaron Reisman" notifications@github.com wrote:

@silvenon https://github.com/silvenon did you figure this out? I am
wondering the same thing.


Reply to this email directly or view it on GitHub
#68 (comment).

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

Successfully merging this pull request may close these issues.

None yet

7 participants