Skip to content

[DNM] WS-2370 - New nav dual script languages#13915

Draft
amoore108 wants to merge 11 commits intolatestfrom
WS-2370-new-nav-dual-script-langs
Draft

[DNM] WS-2370 - New nav dual script languages#13915
amoore108 wants to merge 11 commits intolatestfrom
WS-2370-new-nav-dual-script-langs

Conversation

@amoore108
Copy link
Copy Markdown
Contributor

@amoore108 amoore108 commented Apr 15, 2026

Resolves JIRA: https://bbc.atlassian.net/browse/WS-2370

Summary

Allows dual script services to render the new two-tier navigation

Code changes

  • Updates SERVICES_WITH_NEW_NAV to include dual script services
  • Updates brand SVGs for the dual script services to match designs
  • Removes from Cypress get commands referencing the older navigation layout

Testing

  1. List the steps required to test this PR.

Useful Links

@amoore108 amoore108 self-assigned this Apr 15, 2026
@amoore108 amoore108 changed the title Ws 2370 new nav dual script langs WS-2370 - New nav dual script languages Apr 15, 2026
@amoore108 amoore108 changed the title WS-2370 - New nav dual script languages [DNM] WS-2370 - New nav dual script languages Apr 15, 2026
@amoore108 amoore108 requested a review from Copilot April 15, 2026 12:42
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

Enables dual-script services to participate in the “new nav” rollout and updates their chameleon brand SVGs to match the new navigation designs.

Changes:

  • Adds serbian, ukchina, uzbek, and zhongwen to SERVICES_WITH_NEW_NAV.
  • Updates chameleon logo SVG definitions for the dual-script services (including updated viewbox settings).

Reviewed changes

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

Show a summary per file
File Description
src/app/components/Navigation/config.ts Expands the new-nav allowlist to include dual-script services.
src/app/components/ThemeProvider/chameleonLogos/serbian.tsx Updates Serbian chameleon logo SVG and sets explicit viewbox.
src/app/components/ThemeProvider/chameleonLogos/ukchina.tsx Replaces UK China chameleon logo SVG content and sets explicit viewbox.
src/app/components/ThemeProvider/chameleonLogos/uzbek.tsx Updates Uzbek chameleon logo SVG and sets explicit viewbox.
src/app/components/ThemeProvider/chameleonLogos/zhongwen.tsx Updates Zhongwen chameleon logo SVG and sets explicit viewbox.

Comment thread src/app/components/Navigation/config.ts
Comment thread src/app/components/ThemeProvider/chameleonLogos/ukchina.tsx Outdated
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

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Comment thread src/app/lib/utilities/fetchConfig/index.test.ts
amoore108 and others added 2 commits April 15, 2026 14:10
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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

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

Comment thread src/app/components/ThemeProvider/chameleonLogos/serbian.tsx
Comment thread src/app/components/ThemeProvider/chameleonLogos/ukchina.tsx
Comment thread src/app/components/ThemeProvider/chameleonLogos/zhongwen.tsx
Comment thread src/app/components/ThemeProvider/chameleonLogos/uzbek.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants