Skip to content

feat(cms): Temporarily don't load splitLayout for non-EN locales#19976

Merged
LZoog merged 1 commit intomainfrom
splitlayout-en
Jan 30, 2026
Merged

feat(cms): Temporarily don't load splitLayout for non-EN locales#19976
LZoog merged 1 commit intomainfrom
splitlayout-en

Conversation

@LZoog
Copy link
Copy Markdown
Contributor

@LZoog LZoog commented Jan 30, 2026

Because:

  • We want to do a temporary test of an image that has some text in it that can't be localized (noting, this is not great a11y)

This commit:

  • Loads the normal layout but loads other cms content if the locale is not english

closes FXA-12988


fxa-12988.mov

Because:
* We want to do a temporary test of an image that has some text in it that can't be localized (noting, this is not great a11y)

This commit:
* Loads the normal layout but loads other cms content if the locale is not english

closes FXA-12988
@LZoog LZoog requested a review from a team as a code owner January 30, 2026 19:14
Copy link
Copy Markdown
Contributor

@vpomerleau vpomerleau left a comment

Choose a reason for hiding this comment

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

This works as described, but just noting that we'll need revert and potentially dot release to turn this off since the condition is hard coded. This will also impact all RPs that have splitLayout turned on.

@LZoog LZoog merged commit 0086917 into main Jan 30, 2026
21 checks passed
@LZoog LZoog deleted the splitlayout-en branch January 30, 2026 21:20
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.

2 participants