Closed DFurnes closed 8 years ago
I think this is good to merge in for testing, we may tweak a bit more depending on how it looks on thor/staging.
@weerd @sbsmith86 Ready for another review! Added a type-scale()
function to tidy up where those numbers come from & one unrelated change to the .heading.-banner
pattern.
Nice! Really like the new scale function :dancer: :+1:
Love the type_scale()
stuff! looks good to me ;)
Changes
This PR updates font sizes & line spacing based on new values specified by @lkpttn. It also adds more generous line spacing to wider container elements, and increased spacing between paragraphs.
Screenshots
Here's the pattern library with the old values:![old typography](https://cloud.githubusercontent.com/assets/583202/13998136/e4b02002-f10b-11e5-9110-83c53db4f47f.png)
And with the new values:![new typography](https://cloud.githubusercontent.com/assets/583202/13998137/e7242cf2-f10b-11e5-9bf0-689ac4df9382.png)
...no really, they're different! :mag: :eyes:![animated-2016-03-23_15h-30m-58s](https://cloud.githubusercontent.com/assets/583202/13998185/4b33187a-f10c-11e5-91c5-516ace0707d1.gif)
For review: @DoSomething/front-end