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

Engine init looks up current Sass number precision #1

Merged
merged 1 commit into from
Mar 20, 2018

Conversation

dansobo
Copy link

@dansobo dansobo commented Mar 20, 2018

Sass defaults to 5 digits of precision when performing calculations. This clashes with popular css frameworks like Bootstrap and Foundation. There is ongoing, unresolved discussion as to what defaults to uphold sass/sass#1122.

In the meantime, libsass has made this configurable, but defaults to Sass' values, i.e. sass/libsass#287 and sass/libsass#675

Common workarounds involve editing the precision value directly, i.e. https://github.com/twbs/bootstrap-sass/blame/master/README.md#L78 using the provided accessor http://sass-lang.com/documentation/Sass/Script/Value/Number.html#precision%3D-class_method

This commit simply makes sassc-rails pass along that default value from Sass proper.

Sass defaults to 5 digits of precision when performing calculations. This clashes with popular css frameworks like Bootstrap and Foundation. There is ongoing, unresolved discussion as to what defaults to uphold sass/sass#1122.

In the meantime, libsass has made this configurable, but defaults to Sass' values, i.e. sass/libsass#287 and sass/libsass#675

Common workarounds involve editing the precision value directly, i.e. https://github.com/twbs/bootstrap-sass/blame/master/README.md#L78 using the provided accessor http://sass-lang.com/documentation/Sass/Script/Value/Number.html#precision%3D-class_method

This commit simply makes sassc-rails pass along that default value from Sass proper.
@dansobo dansobo merged commit 0391af8 into atlasobscura:master Mar 20, 2018
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.

2 participants