Skip to content

WS-2562 - Update WS Languages page navigation#14074

Open
louisearchibald wants to merge 37 commits into
latestfrom
WS-2562-update-ws-languages-page-navigation
Open

WS-2562 - Update WS Languages page navigation#14074
louisearchibald wants to merge 37 commits into
latestfrom
WS-2562-update-ws-languages-page-navigation

Conversation

@louisearchibald
Copy link
Copy Markdown
Contributor

@louisearchibald louisearchibald commented May 28, 2026

Resolves JIRA: WS-2562

Summary

Updates the Navigation design on the WS Languages Page only, adding the BBC Blocks to Nav and updating the Brand SVG.

Code changes

  • Updates the colour of item dividers from GREY_10 to GREY_5.
  • Updates the SVG path for ws to show updated design.
  • Adds BBC Blocks to the Navigation Header on the Languages page. (These should link to bbc.com)
  • Uses an isLanguagesPage prop to override legacy styles ensuring no changes to services using the legacy navigation (Hindi) are affected.
  • Adds unit testing for BBC blocks on the languages page.
Screenshot 2026-06-05 at 12 07 37 Screenshot 2026-06-05 at 12 07 52

Testing

  1. Changes can currently be viewed on preview ----> https://simorgh1.belfrage-preview.test.api.bbc.com/ws/languages
  2. Figma designs are here ----> https://www.figma.com/design/dYyWwyRI12jBTEQiLN6qZu/WS-Navigation-update--2026-?node-id=4160-11524&t=eEvS8bE5GFKrXeLc-0

@louisearchibald louisearchibald self-assigned this May 28, 2026
louisearchibald and others added 28 commits May 28, 2026 12:38
…ithub.com/bbc/simorgh into WS-2562-update-ws-languages-page-navigation
…ithub.com/bbc/simorgh into WS-2562-update-ws-languages-page-navigation
…ithub.com/bbc/simorgh into WS-2562-update-ws-languages-page-navigation
…ithub.com/bbc/simorgh into WS-2562-update-ws-languages-page-navigation
…ithub.com/bbc/simorgh into WS-2562-update-ws-languages-page-navigation
…ithub.com/bbc/simorgh into WS-2562-update-ws-languages-page-navigation
…ithub.com/bbc/simorgh into WS-2562-update-ws-languages-page-navigation
@louisearchibald louisearchibald marked this pull request as ready for review June 5, 2026 10:40
Copy link
Copy Markdown
Contributor

@pvaliani pvaliani left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates World Service (WS) Languages page navigation styling and branding, aiming to introduce BBC Blocks in the header and align WS brand assets with the updated design while avoiding impact to services still using legacy navigation.

Changes:

  • Adjusts legacy brand banner/SVG sizing via a new isLanguagesPage prop to support WS Languages-specific layout.
  • Updates WS chameleon logo SVG path data to the revised design and defines an explicit viewbox.
  • Tweaks Collapsible Navigation divider colour from GREY_10 to GREY_5, and updates legacy header logic to show the BBC Blocks banner for WS.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/app/legacy/psammead/psammead-brand/src/index.jsx Adds isLanguagesPage-driven sizing overrides for the brand banner and SVG.
src/app/legacy/containers/Header/index.jsx Expands BBC Blocks banner rendering conditions to include WS.
src/app/legacy/containers/Brand/index.jsx Passes isLanguagesPage down into the legacy Brand to enable WS-specific styling overrides.
src/app/components/ThemeProvider/chameleonLogos/ws.tsx Replaces the WS logo SVG paths and supplies an explicit viewbox.
src/app/components/CollapsibleNavigation/index.styles.ts Updates nav item divider colour token.

Comment thread src/app/legacy/psammead/psammead-brand/src/index.jsx Outdated
Comment thread src/app/legacy/containers/Brand/index.jsx
Comment thread src/app/legacy/containers/Header/index.jsx
Comment thread src/app/legacy/containers/Header/index.jsx Outdated
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.

4 participants