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

Error reading certain values when compiling Bourbon files #380

Closed
wadehammes opened this issue Jun 5, 2014 · 4 comments
Closed

Error reading certain values when compiling Bourbon files #380

wadehammes opened this issue Jun 5, 2014 · 4 comments

Comments

@wadehammes
Copy link

Issue found using following library versions:
Sass v3.3.8
LibSass v2.0
Bourbon v4.0.2

Note: The Bourbon.io GitHub does say to use Bourbon 3.2.x with LibSass, but I think that is because Bourbon 4.0.0 required Sass 3.3.x, which LibSass 2.0 now supports (I'm assuming - if not, remove this issue, the release update noted some support for 3.3.x).

Issue Found

Compiling via Grunt is giving me the following warning, causing Grunt to abort.

Running "sass:dist" (sass) task
Warning: /Users/wade/Dropbox/Code/Personal Dev/Base Wordpress/base-foundation-wordpress-installation/bourbon/helpers/linear-angle-parser:21: error: error reading values after webkit-image
 Use --force to continue.

The line in question within Bourbon inside bourbon/helpers/_linear-angle-parser.scss is:

@return (
      webkit-image: -webkit- + $prefix + ($offset - $num) + $suffix,
      spec-image: $image
);
@akhleung
Copy link

akhleung commented Jun 5, 2014

Hmm, it looks like you're returning a map? Maps aren't implemented in LibSass yet.

@HamptonMakes
Copy link
Member

We're hoping to get Bourbon compat back in the next release. We know that
we don't support maps yet.

On Thu, Jun 5, 2014 at 8:31 PM, Aaron Leung notifications@github.com
wrote:

Hmm, it looks like you're returning a map? Maps aren't implemented in
LibSass yet.

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

@wadehammes
Copy link
Author

@akhleung @hcatlin that is what I thought, honestly not completely up-to-date on maps yet, so wasn't really sure what this issue pertained to. Thanks for the clarification, and love all the new stuff!

@akhleung
Copy link

akhleung commented Jun 6, 2014

Closing this since it's covered by #263.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants