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

Forbid imports deeply nested in control directives #3097

Open
nex3 opened this issue May 14, 2020 · 1 comment
Open

Forbid imports deeply nested in control directives #3097

nex3 opened this issue May 14, 2020 · 1 comment
Labels
Compatibility - P1 ⚠️ Very important for compatibility with the Sass spec and ecosystem

Comments

@nex3
Copy link
Contributor

nex3 commented May 14, 2020

LibSass currently allows imports to be nested in control directives as long as they're also nested within style rules. For example:

@if true {
  a {@import 'foo'}
}

This is invalid Sass and should produce an error. Note that the following does produce an error:

@if true {
  @import 'foo';
}
@nex3 nex3 added the Compatibility - P1 ⚠️ Very important for compatibility with the Sass spec and ecosystem label May 14, 2020
@vrushank
Copy link

Any progress on this one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Compatibility - P1 ⚠️ Very important for compatibility with the Sass spec and ecosystem
Projects
None yet
Development

No branches or pull requests

2 participants