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

Variable in to SCSS code #275

Closed
Hooter opened this issue Jul 29, 2014 · 4 comments

Comments

@Hooter
Copy link

commented Jul 29, 2014

Simple SCSS code:

/*colors*/
$color_0: #fff;
$color_1: #285592;
$color_2: #86a9c2;

or

/*fonts*/
$font_0: verdana;
$font_1: sans-serif;

Result in to console:

stack:  d:\tmp\scss\err\/buttons.scss
Please check the validity of the block starting from line #3

1 | /*colors*/
2 | $color_0: #fff;
3*| $color_1: #285592;
4 | $color_2: #86a9c2;

Gonzales PE version: 3.0.0-10
CSScomb Core version: 2.0.2

@tonyganch tonyganch added bug labels Jul 31, 2014

@tonyganch tonyganch added this to the Next release milestone Jul 31, 2014

@stephenbe

This comment has been minimized.

Copy link

commented Oct 1, 2014

Also a dash does not work :

Will not compile

$color-1d: red;

will compile

$color-d: red;
@cmnstmntmn

This comment has been minimized.

Copy link

commented Dec 12, 2014

also local variables are added at the end of element. so there will be no visibility for them

footer {
  $text-color: red;
  $text-spacing: 2px;

  color: $text-color;

    a {
        color:transparentize($text-color, .5)
    }
  }

post edit. it seemns that adding to the CSScomb.sublime-settings, fix this issue

        "sort-order": [
        [
            "$variable"
        ],
        ...
@vxsx

This comment has been minimized.

Copy link

commented Dec 29, 2014

also breaks on extends with classes which have numeric values

stack:  private/sass/addons/_accordion.scss
Please check the validity of the block starting from line #34

32 |             background: white;
33 |             @include border-radius(100%);
34*|             @extend .icon-rotate-180;
35 |         }
36 |         // collapsed state

if I remove -180 it works correctly

@tonyganch

This comment has been minimized.

Copy link
Member

commented Jun 9, 2015

Fixed in csscomb@3.1.7

@tonyganch tonyganch closed this Jun 9, 2015

@sodatea sodatea modified the milestone: Next release Jan 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.