-
Notifications
You must be signed in to change notification settings - Fork 119
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 escape e() causes problems #40
Comments
I'm pretty sure including the raw css within the e() function will include it - though I can't find reference material to confirm this. On that note, I'm having issues with this too (haven't tried fixing it yet) |
for a very quick fix, you can implement the missing function:
|
The thing is that the function isn't missing, but it was introduced in compass master a few days ago, and a new version of compass gem was pushed yesterday, and hence people are getting this error now. I tracked it down to this commit in compass: I guess namespacing or renaming e() should suffice. |
I also had this problem. |
Hi, i think its safe to remove the e() from the compass version because e seems to be a escaping function for javascript based less variant but we dont need them on compass/sass/scss side so we can remove theme safetly here ?! |
Agree with @blackice2999, you shouldn't need to escape in sass. |
I sent a pull request that does just that. |
Since it is fixed in c4155a3 this issue can be closed, I guess. When will a new gem be pushed? |
Seems like this has been addressed, and this should be closed? |
Using version: 2.0.1.2
I'm getting an error:
Sass::SyntaxError: wrong number of arguments (1 for 0) for `e' (in app/assets/stylesheets/base.css.sass)
/Volumes/Clank/Users/Mike/.rvm/gems/ruby-1.9.2-p290@socialply/gems/compass_twitter_bootstrap-2.0.1.2/stylesheets/compass_twitter_bootstrap/_mixins.scss:540:in `buttonBackground'
Browsing the file, I see 3 or so instances of a mixin: e(). Doing a search reveals that this is a less function for escaping the content. I'm not sure what the correct course of action is, but I commented out each use of e() to get working.
The text was updated successfully, but these errors were encountered: