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

Can't compile bigfoot-number.scss #13

Closed
philgruneich opened this issue Mar 16, 2014 · 8 comments
Closed

Can't compile bigfoot-number.scss #13

philgruneich opened this issue Mar 16, 2014 · 8 comments

Comments

@philgruneich
Copy link

CodeKit 2 is returning the following error:

Compass was unable to compile one or more files in the project:

error style.scss (Line 40 of _bigfoot-number.scss: List index is 2 but list is only 1 item long for `nth')
@philgruneich
Copy link
Author

This is line 40:

$new-props: $new-props + ", " + property-checker(nth($properties, $i), $prefix);

from:

@function prefix-properties($properties, $prefix) {
    $new-props: property-checker(nth($properties, 1), $prefix);
    @for $i from 2 through length($properties) {
        $new-props: $new-props + ", " + property-checker(nth($properties, $i), $prefix);
    }
    @return unquote($new-props);

@philgruneich
Copy link
Author

If I change the function to:

@function prefix-properties($properties, $prefix) {
    $new-props: property-checker(nth($properties, 1), $prefix);
    @for $i from 1 through length($properties) {
        $new-props: $new-props + ", " + property-checker(nth($properties, $i), $prefix);
    }
    @return unquote($new-props);
}

It works, could you help me debug to see if it breaks anything?

@lemonmade
Copy link
Owner

In the compiled CSS, does it not produce a duplicate declaration of whatever the first property is?

@philgruneich
Copy link
Author

Not so sure what am I looking for, Chris, but everything seems to be alright after changing the integer. This is the compiled css file: http://philgr.com/assets/styles/v2_style.css

@lemonmade
Copy link
Owner

Can you show me the @include line in your Sass file that ends up using this function? I can't reproduce this issue.

@philgruneich
Copy link
Author

These are the candidates:

Line 234
@include transition-property(opacity);
Line 533
@include transition-property($popover-transition);
Line 576
@include transition-property(transform);
Line 628
@include transition-property($popover-scroll-indicator-transition);
Line 717
@include transition-property($popover-main-wrapper-transition);

@lemonmade
Copy link
Owner

Can you see if c3b0a67 still has these issues?

@philgruneich
Copy link
Author

Works great. Thank you (:

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