Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WSTEAM1-1005: Gujarati Telugu font updates #11599

Merged
merged 14 commits into from May 23, 2024

Conversation

Isabella-Mitchell
Copy link
Contributor

@Isabella-Mitchell Isabella-Mitchell commented May 7, 2024

Resolves JIRA WSTEAM1-1005 - Simorgh Changes, subtask of WSTEAM1-900 - Updated fonts for two Indian services

Overall changes

Updates default fonts for two services
Telugu from 'Mallanna' to 'Noto Sans Telugu'
Gujarati from 'Mukta Vaani' to 'Noto Sans Gujarati'

Code changes

STORYBOOK

  • Updates Telugu font in storybook
  • Adds Gujarati font to Storybook (I think because the previous one wasn't embedded, I figured it would be useful to add)
  • Adds both in regular and bold

THEME PROVIDER

  • Updates Telugu fontFaces
  • Adds Gujarati fontfaces (I think because the previous one wasn't embedded, I figured it was necessary to add)
  • Adds both in regular and bold (previously Telugu only had regular - figured it would be useful to add bold, but can remove if it's an issue)
  • Update Gujarati and Telugu fontFamilies as per UX comments in the linked subtasks

PSAMMEAD

  • Update legacy by updating psammead font-families.js
  • Update snapshots

Testing

  1. Can preview on Storybook.

I think UX can test this on preview. Otherwise we can put a change freeze in and test on test.

Helpful Links

Dropbox paper instructions

Coding Standards

Repository use guidelines

const MALLANA_FONTS_DIR =
'https://ws-downloads.files.bbci.co.uk/fonts/Mallanna/v1.0.4/';
const NOTO_SANS_TELUGU_FONTS_DIR =
'https://ws-downloads.files.bbci.co.uk/fonts/NotoSansTelugu/v1.00/';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not live yet, hence 'do not merge' label

'https://ws-downloads.files.bbci.co.uk/fonts/NotoSansTelugu/v1.00/';

const NOTO_SANS_GUJARATI_FONTS_DIR =
'https://ws-downloads.files.bbci.co.uk/fonts/NotoSansGujarati/v1.00/';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not live yet, hence 'do not merge' label

Copy link
Contributor

Choose a reason for hiding this comment

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

is there a PR for the font server repo or is that already up there/has it been merged?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep it's linked in this ticket: https://jira.dev.bbc.co.uk/browse/WSTEAM1-1003

It has been merged but I'm still working through the steps to deploy

@Isabella-Mitchell Isabella-Mitchell marked this pull request as ready for review May 8, 2024 15:45
@pvaliani
Copy link
Contributor

pvaliani commented May 9, 2024

Re-running the E2E's on this but Cypress just got patched this morning:
#11605

And i'm not sure if the failure is related, just FYI.

@Isabella-Mitchell Isabella-Mitchell merged commit 289302e into latest May 23, 2024
11 checks passed
@Isabella-Mitchell Isabella-Mitchell deleted the WSTEAM1-1005-gujarati-telugu-fonts branch May 23, 2024 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants