Skip to content
This repository has been archived by the owner on Mar 31, 2020. It is now read-only.

Use precision of 10 by default for SASS task #308

Merged
merged 1 commit into from
Oct 13, 2015
Merged

Use precision of 10 by default for SASS task #308

merged 1 commit into from
Oct 13, 2015

Conversation

jwalton512
Copy link
Contributor

The default in SASS is 5. Bootstrap (bootstrap-sass included with laravel by default) utilizes a $line-height-base of 1.428571429. The default precision rounds this to 1.42857. This doesn't cause many noticeable issues, but does cause input groups with buttons (input-group-btn) to be misaligned. Issue is described here

The default in SASS is 5.  Bootstrap (bootstrap-sass included with laravel by default) utilizes a `$line-height-base` of `1.428571429`.  The default precision rounds this to `1.42857`.  This doesn't cause many noticeable issues, but does cause input groups with buttons (`input-group-btn`) to be misaligned.  Issue is described [here](twbs/bootstrap-sass#409)
@jwalton512
Copy link
Contributor Author

In the interim, as a workaround, the code below can be added into gulpfile.js

elixir.config.css.sass.pluginOptions.precision = 10;

JeffreyWay added a commit that referenced this pull request Oct 13, 2015
Use precision of 10 by default for SASS task
@JeffreyWay JeffreyWay merged commit 668a79d into laravel:master Oct 13, 2015
@JeffreyWay
Copy link
Contributor

Thanks!

@jwalton512 jwalton512 deleted the patch-1 branch October 13, 2015 18:39
@jwalton512
Copy link
Contributor Author

@JeffreyWay any way this can be merged into 3.0.0? or tag a 3.0.1?

@JeffreyWay
Copy link
Contributor

Okay, tagged it just now. You can npm update.

@jwalton512
Copy link
Contributor Author

awesome, thanks @JeffreyWay

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

Successfully merging this pull request may close these issues.

2 participants