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.

Sign up for GitHub

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

Jump to bottom

calc() expression reducer #10

Open
c9s opened this issue May 1, 2015 · 1 comment
Open

calc() expression reducer #10

c9s opened this issue May 1, 2015 · 1 comment
Labels
Improvement SASS Spec
Milestone
Pre-Release

Comments

@c9s
Copy link
Owner

c9s commented May 1, 2015

While SASS is interpreing this as it is:

$some: 22;
width: calc(100% - 10px + 20px + $some);

We can reduce the expression if they are the same unit to:

width: calc(100% + 10px + 22);
@c9s c9s added the Improvement label May 1, 2015
@c9s c9s modified the milestone: Pre-Release May 2, 2015
@c9s c9s added the CSS2.1 Spec label May 6, 2015
@c9s c9s mentioned this issue May 8, 2015
Closed
@c9s c9s added SASS Spec and removed CSS2.1 Spec labels May 9, 2015
@c9s
Copy link
Owner Author

c9s commented May 19, 2015

Add FunctionCall.IsLiteral() method to check if we need to present the function call as literal string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Improvement SASS Spec
Projects
None yet
Milestone
Pre-Release
Development

No branches or pull requests
1 participant
@c9s