Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Update Headline & Subheading colour to Shadow #305

Merged
merged 7 commits into from
Feb 27, 2019
Merged

Conversation

sareh
Copy link
Contributor

@sareh sareh commented Feb 25, 2019

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

Subheading component shows color is Shadow and font updated to Reith Sans Bold:
screen shot of subheading


  • I have assigned myself to this PR and the corresponding issues
  • Tests added for new features
  • Test engineer approval

@sareh sareh self-assigned this Feb 25, 2019
@sareh sareh requested a review from a team as a code owner February 25, 2019 09:19
Copy link
Contributor

@dr3 dr3 left a 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?

@sareh
Copy link
Contributor Author

sareh commented Feb 25, 2019

@dr3 Yes. I've pushed that change.

dr3
dr3 previously approved these changes Feb 25, 2019
Copy link
Contributor

@dr3 dr3 left a 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 {
Copy link
Contributor Author

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.

.storybook/config.js Outdated Show resolved Hide resolved
@sareh sareh requested review from dr3, bcmn and ChrisBAshton February 26, 2019 14:17
Copy link
Contributor

@bcmn bcmn left a 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?

@sareh
Copy link
Contributor Author

sareh commented Feb 26, 2019

@bcmn Thanks. I've opened this issue to capture this work: #312

@jamesbrumpton
Copy link
Contributor

👍

@sareh sareh merged commit 08d9196 into latest Feb 27, 2019
@sareh sareh deleted the headings-update branch February 27, 2019 10:26
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.

4 participants