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

scss language addition #192

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

scss language addition #192

wants to merge 14 commits into from

Conversation

FabioDiMonte
Copy link

Added SCSS syntax recognition, which includes:

  • all features from PR css language enhancements #191
  • variables (eg: $myVar )
  • interpolations (eg: #{$myClass} )
  • all SCSS at-directives (such as @mixin , @function , @extend , @import , @include ) and all their inner syntax
  • all loops (ie: @for , @each and @while ) along with their inner syntax
  • conditions (both if/else and ternary)
  • scss lists and maps

@FabioDiMonte FabioDiMonte changed the title Feature/scss language addition scss language addition Apr 22, 2016
@FabioDiMonte
Copy link
Author

some screenshots made using the code in this PR (including the changed CSS theme "solarized dark")

base scss syntax (nesting, placeholders, mixins, ...)

nesting placeholders mixins

loops, conditions, interpolations

loops and interpolations

conditions interpolations

interpolations

lists and maps

lists and maps


'use strict';

if(!Rainbow){
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why would Rainbow not be included here? Also checking for it directly will actually trigger an error if it doesn't exist

Copy link
Author

@FabioDiMonte FabioDiMonte Apr 22, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agree.. will change that soon (also on the other PRs, #191 and #193 )

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

Successfully merging this pull request may close these issues.

2 participants