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

Differentiate CSS import and Sass imports #2096

Open
xzyfer opened this issue May 26, 2016 · 0 comments
Open

Differentiate CSS import and Sass imports #2096

xzyfer opened this issue May 26, 2016 · 0 comments

Comments

@xzyfer
Copy link
Contributor

xzyfer commented May 26, 2016

The follow Sass should not error because the @import is a CSS import not a Sass import. This was previously fixed in #1724 but has since regressed.

@mixin foo() {
  @import "https://foo";
}
@include foo;

LibSass 3.3.6

Error: Import directives may not be used within control directives or mixins.
        on line 2 of test.scss
>>   @import "https://foo";
   --^

Ruby Sass 3.4.21

@import "https://foo";

Spec added sass/sass-spec#910

@xzyfer xzyfer added this to the 3.4 milestone May 26, 2016
xzyfer added a commit to xzyfer/sass-spec that referenced this issue Sep 8, 2016
@xzyfer xzyfer modified the milestones: 3.4.1, 3.4 Sep 8, 2016
@xzyfer xzyfer modified the milestones: 3.4.1, 3.4.x Dec 28, 2016
@nex3 nex3 added the Compatibility - P3 Minorly important for compatibility with the Sass spec and ecosystem label Jun 3, 2019
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