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

Sass Scanner #93

Closed
nathany opened this issue Oct 28, 2012 · 7 comments
Closed

Sass Scanner #93

nathany opened this issue Oct 28, 2012 · 7 comments
Assignees
Milestone

Comments

@nathany
Copy link
Contributor

nathany commented Oct 28, 2012

Syntactically Awesome Style Sheets come in two formats: the whitespace significant original (sass) and the popular CSS-derived alternative (scss). This may require 2 separate scanners, though they share the same keywords, etc.

Reference:

There are a number of SASS mixin libraries for sample code:

@korny
Copy link
Member

korny commented Jun 10, 2013

Sass has been added.

@korny
Copy link
Member

korny commented Jun 10, 2013

Can I refuse to write a SCSS scanner? The http://thesassway.com/articles/sass-vs-scss-which-syntax-is-better discussion is really the pinnacle of meaningful-whitespace-phobia :(

@nathany
Copy link
Contributor Author

nathany commented Jun 10, 2013

Hah, well I suppose you can, but I think SCSS is a little more popular than the older whitespace SASS format.

@korny
Copy link
Member

korny commented Jun 10, 2013

I think SCSS is a little more popular than the older whitespace SASS format.

True. But the Sass syntax is by no means "old", it's just too modern for design folks.

Should be easier to parse though :-)

@nathany
Copy link
Contributor Author

nathany commented Jun 10, 2013

I'm with you on preferring SASS. Especially the lack of semicolons.

Though shouldn't SCSS be similar to CSS to parse?

@korny
Copy link
Member

korny commented Jun 10, 2013

Yes, it's essentially CSS plus the new Sass tokens. I'm trying to re-use as much as possible, but with CodeRay's method-based scanners that still means a lot of duplication.

@korny
Copy link
Member

korny commented Jun 11, 2013

Creating a new ticket for the SCSS scanner. Thank you for your support, @nathany :)

@korny korny closed this as completed Jun 11, 2013
@korny korny mentioned this issue Jun 11, 2013
@ghost ghost assigned korny Jun 11, 2013
@thybzi thybzi mentioned this issue Jul 26, 2013
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

2 participants