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

`--math` AKA align strict math behavior with documentation. #3273

Closed

calvinjuarez opened this issue Jul 7, 2018 · 2 comments

Member

calvinjuarez commented Jul 7, 2018

Per a gitter discussion:

--math=(always|no-division|parens|strict|strict-legacy)
-m=...

always:

height: (5px + 5px) + 2px; → height: 12px;
width: 10px + 1; → width: 11px;
width: (10px + 1); → width: 11px;
font: 16 / 4; → font: 4;
font: (16 / 4); → font: 4;

no-division:

height: (5px + 5px) + 2px; → height: 12px;
width: 10px + 1; → width: 11px;
width: (10px + 1); → width: 11px;
font: 16 / 4; → font: 16 / 4;
font: (16 / 4); → font: 4;

parens AND strict:

height: (5px + 5px) + 2px; → height: 10px + 2px;
width: 10px + 1; → width: 10px + 1;
width: (10px + 1); → width: 11px;
font: 16 / 4; → font: 16 / 4;
font: (16 / 4); → font: 4;

strict-legacy:

height: (5px + 5px) + 2px; → height: (5px + 5px) + 2px;
width: 10px + 1; → width: 10px + 1;
width: (10px + 1); → width: 11px;
font: 16 / 4; → font: 16 / 4;
font: (16 / 4); → font: 4;

Some Questions to Consider:

What about calc()?

The text was updated successfully, but these errors were encountered:

Member

matthew-dean commented Jul 7, 2018

@calvinjuarez I went with parens-all and parens-division - see PR - #3274

Member

matthew-dean commented Jul 7, 2018 •

edited

Loading

Closing just so we can keep the discussion in one place (in PR)

matthew-dean closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment