-
Notifications
You must be signed in to change notification settings - Fork 464
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
Error parsing selector with escaped colon at the end #2625
Comments
My gut reaction is that this is not valid CSS syntax. If this word
previously it may have been a bug that was patched. I'll confirm when I'm
at a keyboard.
…On Thu., 29 Mar. 2018, 9:32 pm James Wilson, ***@***.***> wrote:
Using libsass 3.5.2 (via node-sass 4.8.3), The following leads to an error:
$ echo -e 'something\:{ padding: 2px; }' | ./node-sass
{
"status": 1,
"file": "stdin",
"line": 1,
"column": 1,
"message": "Invalid CSS after \"s\": expected 1 selector or at-rule, was \"something\\\\:{ paddin\"",
"formatted": "Error: Invalid CSS after \"s\": expected 1 selector or at-rule, was \"something\\\\:{ paddin\"\n on line 1 of stdin\n>> something\\:{ padding: 2px; }\n\n ^\n"
}
Adding a space between the colon and opening curly bracket works, eg:
$ echo -e 'something\: { padding: 2px; }' | ./node-sass
something\: {
padding: 2px; }
This is a regression, and does not appear to be a problem using libsass
3.5.0.beta.2 (via node-sass 4.5.3).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#2625>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAjZWDVQLATnJQEr_2uRmWrpa_dXTBJ6ks5tjLhKgaJpZM4TAHEw>
.
|
Confirmed this is a regression in 3.5.0. |
xzyfer
added a commit
to xzyfer/sass-spec
that referenced
this issue
Mar 30, 2018
This PR activates specs for sass/libsass#2625
xzyfer
added a commit
to xzyfer/libsass
that referenced
this issue
Mar 30, 2018
xzyfer
added a commit
to xzyfer/libsass
that referenced
this issue
Mar 30, 2018
xzyfer
added a commit
that referenced
this issue
Mar 30, 2018
xzyfer
added a commit
that referenced
this issue
Mar 30, 2018
xzyfer
added a commit
to sass/sass-spec
that referenced
this issue
Mar 30, 2018
This PR activates specs for sass/libsass#2625
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Using libsass 3.5.2 (via node-sass 4.8.3), The following leads to an error:
Adding a space between the colon and opening curly bracket works, eg:
This is a regression, and does not appear to be a problem using libsass 3.5.0.beta.2 (via node-sass 4.5.3).
The text was updated successfully, but these errors were encountered: