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

Feature: add precision parameter #39

Merged
merged 3 commits into from Jan 7, 2015
Merged

Feature: add precision parameter #39

merged 3 commits into from Jan 7, 2015

Conversation

astagi
Copy link
Contributor

@astagi astagi commented Jan 5, 2015

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.15%) when pulling f4d1494 on nephila:feature/precision into 5e924e6 on dahlia:python.

@dahlia
Copy link
Member

dahlia commented Jan 6, 2015

Thanks for your effort! Could you write some docs about the parameter? It wouldn’t only be helpful for libsass-python users, but help me, the maintainer who should review this patch, to understand what is the parameter for as well!

@coveralls
Copy link

Coverage Status

Coverage increased (+0.15%) when pulling 29998e1 on nephila:feature/precision into 5e924e6 on dahlia:python.

@astagi
Copy link
Contributor Author

astagi commented Jan 6, 2015

No problem :) Latest commit adds some doc. "precision" parameter set the precision for numbers, for example if you specify precision=8 you'll get numbers with a precision of 8 decimal places:

$font-stack:    Helvetica, sans-serif;
$primary-color: #333;
$variabile: 5 / 3 * 6 / 7;
body {
    font: 100% $font-stack;
    color: $primary-color;
    height: $variabile;
}

the compiled css will result:

body {
    font: 100% Helvetica, sans-serif;
    color: #333;
    height: 1.42857143; }

With precision = 5, "height" would be equal to 1.42857

@dahlia dahlia merged commit 29998e1 into sass:python Jan 7, 2015
dahlia added a commit that referenced this pull request Jan 7, 2015
Feature: add precision parameter
@dahlia
Copy link
Member

dahlia commented Jan 7, 2015

Merged!

@astagi
Copy link
Contributor Author

astagi commented Jan 8, 2015

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants