search

Financial-Times / o-typography

Typography and vertical rhythm styles for FT branding
http://registry.origami.ft.com/components/o-typography
10 stars 2 forks source link

Fix progressive loading of the display font. #147

Closed notlee closed 6 years ago

notlee commented 6 years ago

This updates display labels to match those used in the css https://github.com/Financial-Times/o-typography/blob/master/src/scss/_type-mixins.scss#L36

It looks like its been this way for a while: https://github.com/Financial-Times/o-typography/commit/cd7bb58f07676e0c64d71adf771847610e527390#diff-cc5f92b397da4aef0ff4fed64a71a7a4L410

o-typography logic is duplicated in n-ui. Will submit a separate PR there. https://github.com/Financial-Times/n-ui/blob/b08b20bf757bab1103f09adfecb22a90feebca36/browser/layout/partials/enhance-fonts.html#L3

Before (core)

screen shot 2018-05-24 at 17 11 53

After (core)

screen shot 2018-05-24 at 17 11 58