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

Talos - Bump @bbc/psammead-styles #1734

Merged
merged 11 commits into from
Aug 14, 2019
Merged

Talos - Bump @bbc/psammead-styles #1734

merged 11 commits into from
Aug 14, 2019

Conversation

bbc-news-frameworks
Copy link
Contributor

@bbc-news-frameworks bbc-news-frameworks commented Aug 13, 2019

👋 The following packages have been published:
@bbc/psammead-styles

So we need to bump them in the following packages:
@bbc/psammead-consent-banner
@bbc/psammead-copyright
@bbc/psammead-image-placeholder
@bbc/psammead-headings
@bbc/psammead-navigation
@bbc/psammead-sitewide-links
@bbc/psammead-brand
@bbc/psammead-paragraph
@bbc/psammead-caption
@bbc/psammead-story-promo-list
@bbc/psammead-inline-link
@bbc/psammead-story-promo
@bbc/psammead-media-indicator
@bbc/psammead-timestamp
@bbc/psammead-section-label


Testing note added by @sareh: @bbc/[email protected] fixed the font base url for Latha (used by the Tamil service) and so it should be bumped up through all of these packages, so that in our consuming application Simorgh, we don't have multiple versions of psammead-styles.

Also updated to include bumps to these packages:
The following package has been published:
@bbc/gel-foundations

So we need to bump them in the following packages:
psammead-brand
psammead-caption
psammead-consent-banner
psammead-copyright
psammead-headings
psammead-media-indicator
psammead-navigation
psammead-paragraph
psammead-section-label
psammead-sitewide-links
psammead-story-promo-list
psammead-story-promo
psammead-timestamp
psammead-storybook-helpers

Copy link
Contributor

@sareh sareh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, Talos. :D (@dr3)

@dr3
Copy link
Contributor

dr3 commented Aug 14, 2019

This may need to wait for talos to be merged, so this PR can become based off of latest

@dr3 dr3 changed the base branch from BumperBotIntegrate-new-new-new-new-new to latest August 14, 2019 09:44
@dr3 dr3 mentioned this pull request Aug 14, 2019
@paruchurisilpa paruchurisilpa self-assigned this Aug 14, 2019
@sareh
Copy link
Contributor

sareh commented Aug 14, 2019

Changes that were bumped here in this PR have been UX reviewed.

  • @bbc/psammead-styles bump was to fix the base font url for Latha
  • @bbc/gel-foundations bump included Thai type sizes updates for Brevier and Minion (used by sitewide links, media indicator and timestamps)

@sareh
Copy link
Contributor

sareh commented Aug 14, 2019

Update to sitewide links to use the new Thai typography settings:
http://localhost:8180/iframe.html?id=components-sitewidelinks--default&knob-Select%20a%20service=thai

Thai SitewideLinks Font size 14px, line height 18px
Screen Shot Thai sitewide links

Thai Media indicator text has font-size 12px, line height 16px.
Screen Shot Media indicator Thai

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

Successfully merging this pull request may close these issues.

5 participants