-
Notifications
You must be signed in to change notification settings - Fork 54
Update Headline & Subheading colour to Shadow #305
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this PR resolve "Change subheadings in psammead-headings to use Reith Sans Bold
." from the issue #279?
@dr3 Yes. I've pushed that change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -43,6 +43,13 @@ const GlobalStyle = createGlobalStyle` | |||
font-weight: 600; | |||
src: url('https://gel.files.bbci.co.uk/r2.302/BBCReithSerif_W_Md.woff2') format('woff2'), url('https://gel.files.bbci.co.uk/r2.302/BBCReithSerif_W_Md.woff') format('woff'); | |||
} | |||
@font-face { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this so the Reith Sans Bold font will load in Storybook. Though issue #285 updating font-faces within Psammead utilities will be used to update the font-faces in the Storybook config.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good in Storybook -- slightly concerned about making sure we guarantee we've updated the bold to be correct following the proper implementation, though. Is there an issue for this, @sareh?
👍 |
Resolves #279
Overall change: Update Headline & Subheading colour to Shadow
'#3F3F42'
& Subheading updated to use Reith Sans Bold font.Screenshots of Storybook for Headline, with dev tools showing color is Shadow & font is unchanged:
![screen shot of heading](https://user-images.githubusercontent.com/3028997/53327303-7008d400-38df-11e9-9e18-a18058a615cb.png)
Subheading component shows color is Shadow and font updated to Reith Sans Bold:
![screen shot of subheading](https://user-images.githubusercontent.com/3028997/53330144-ab0e0600-38e5-11e9-9a35-62fead502e3b.png)