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

Space After Colons doesn't work inside of arguments #98

Closed
Snugug opened this issue Sep 4, 2015 · 5 comments
Closed

Space After Colons doesn't work inside of arguments #98

Snugug opened this issue Sep 4, 2015 · 5 comments
Milestone

Comments

@Snugug
Copy link
Member

Snugug commented Sep 4, 2015

For instance, inside of Media Query arguments:

bgriffith@2ceb329#diff-9749c71a6762edb9480fb15c79950231R23

@Snugug Snugug added this to the 1.2.0 milestone Sep 4, 2015
@bgriffith
Copy link
Member

Had a quick look and the issue is that colons inside a media query aren't classified as a propertyDelimiter so it doesn't include them in the loop.

@DanPurdy
Copy link
Member

DanPurdy commented Sep 4, 2015

What are they classed as?

@bgriffith
Copy link
Member

Just operators

{
                      "type": "operator",
                      "content": ":",
                      "syntax": "scss",
                      "start": {
                        "line": 2,
                        "column": 20
                      },
                      "end": {
                        "line": 2,
                        "column": 20
                      }
                    }

@bgriffith
Copy link
Member

@Snugug Shouldn't the title of this be 'Space after colon..? instead of parens?

@Snugug Snugug changed the title Space After Parens doesn't work inside of arguments Space After Colons doesn't work inside of arguments Sep 4, 2015
@Snugug
Copy link
Member Author

Snugug commented Sep 4, 2015

Yah… probably…

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