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

Properly compute standard property for nested SCSS properties so that rules such as experimental-property-usage works fine #6

Closed
hcomnetworkers opened this issue Apr 25, 2017 · 1 comment
Labels
Milestone

Comments

@hcomnetworkers
Copy link

Hi,

I get the violation 'Remove this usage of the experimental "size" property.' on a file but it seems to be wrong in this case.

Here is the rule in my .scss file:

@mixin font-normal {
  color: $color-font;
  font: {
    family: Arial, Helvetica, sans-serif;
    size: $fontSize-m;
    weight: 300;
  }
  line-height: $lineHeight-m;
}

The violation is right in the row containing the size.

It seems that the outer font is not detected here but it is valid CSS.
It is quite interesting that the other rules family and weight got no violation :-)

Is it possible to adjust the rule?

Thanks in advance!

@racodond
Copy link
Owner

Thanks for your feedback! I'll get back to you when I get back home beginning of next week.

@racodond racodond added this to the 4.1 milestone Apr 25, 2017
@racodond racodond added the bug label May 3, 2017
@racodond racodond changed the title Property "size" in font definition not detected in scss file Properly compute standard property for nested SCSS properties so that rules such as experimental-property-usage works fine May 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants