Skip to content

Full rollout of revamped immersive layout behind A/B test - #16621

Merged
frederickobrien merged 6 commits into
mainfrom
revamped-immersives-everywhere
Sep 2, 2026
Merged

Full rollout of revamped immersive layout behind A/B test#16621
frederickobrien merged 6 commits into
mainfrom
revamped-immersives-everywhere

Conversation

@frederickobrien

@frederickobrien frederickobrien commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Rather than getting into a rebase nightmare in #15922 following #16506, this is a clean run at a wider rollout of the new immersive designs.

This pull request:

  • Puts the changes behind a 0% test so we can get the changes out the door and test in the real world
  • Sends all immersives down the new grid-based system for those who have opted in to the test
  • Applies further adjustments and fixes to the new immersive layout, including ripping out now redundant forks and styling for feature versions of the landscape and portrait immersive layouts

Add ?ab-articles-and-publishing-revamped-immersive-layout=enable to the end of a Guardian url to opt in.

Opt in and go to an immersive piece and you should be met with the new look:

image

Rather than the old one:

image

We can follow up on this pull request with further adjustments and fixes until we're happy to roll out the change for real. At that point we can delete the old ImmersiveLayout entirely and get a drink to celebrate.

@frederickobrien frederickobrien added the feature Departmental tracking: work on a new feature label Aug 27, 2026
@frederickobrien frederickobrien added this to the Interactives milestone Aug 27, 2026
@frederickobrien
frederickobrien force-pushed the revamped-immersives-everywhere branch from d8d92ab to 708c784 Compare August 27, 2026 11:11
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

🚀 Image pushed to AWS ECR

Image digest: sha256:42a5623bf2d2ae8c5152ea7d6c5f326bbbbfc84a2fa8bf8de72d1bb81bde2586

🐛 Run the image locally

The following can be used to run the image locally:

# Refer to image using the immutable digest. Find alternatives below.
IMAGE_IDENTIFIER="@sha256:42a5623bf2d2ae8c5152ea7d6c5f326bbbbfc84a2fa8bf8de72d1bb81bde2586"

# Refer to image using branch tag
# IMAGE_IDENTIFIER=":branch-revamped-immersives-everywhere"

# Refer to image using build tag
# IMAGE_IDENTIFIER=":build-30672"

# Refer to image via the GitHub commit SHA tag
# IMAGE_IDENTIFIER=":sha-fc904a26d324147c1fe5b305baae145b7c56c096"

# Set environment variables for the AWS CLI
AWS_PROFILE="<A_PROFILE_FROM_JANUS>"
AWS_DEFAULT_REGION="eu-west-1"

IMAGE_ACCOUNT_ID=$(aws ssm get-parameter --name /organisation/accounts/deployTools --query "Parameter.Value" --output text)
REGISTRY="${IMAGE_ACCOUNT_ID}.dkr.ecr.${AWS_DEFAULT_REGION}.amazonaws.com"
IMAGE="${REGISTRY}/guardian/dotcom-rendering${IMAGE_IDENTIFIER}"

# Login to AWS ECR https://docs.aws.amazon.com/AmazonECR/latest/userguide/registry_auth.html
aws ecr get-login-password | docker login --username AWS --password-stdin $REGISTRY

# Pull the image
docker pull $IMAGE

# Run the image. You'll likely need to set additional flags. See https://docs.docker.com/reference/cli/docker/container/run.
docker run $IMAGE

From guardian/actions-publish-image.

@frederickobrien
frederickobrien force-pushed the revamped-immersives-everywhere branch from 708c784 to ef3998a Compare August 27, 2026 11:36
@frederickobrien frederickobrien changed the title Full rollout of revamped immersive layout Full rollout of revamped immersive layout behind A/B test Aug 28, 2026
@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown

@frederickobrien
frederickobrien force-pushed the revamped-immersives-everywhere branch from 5472a21 to 2e002dc Compare August 28, 2026 16:12
@frederickobrien
frederickobrien requested a review from a team September 1, 2026 08:39
@frederickobrien
frederickobrien marked this pull request as ready for review September 1, 2026 08:44
@frederickobrien
frederickobrien requested a review from a team as a code owner September 1, 2026 08:44
@frederickobrien
frederickobrien force-pushed the revamped-immersives-everywhere branch from 2e002dc to 0497cbf Compare September 1, 2026 08:44
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@frederickobrien frederickobrien added the run_chromatic Runs chromatic when label is applied label Sep 1, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Sep 1, 2026
Copilot AI requested a review from cemms1 September 1, 2026 09:30
Comment thread dotcom-rendering/playwright/tests/commercial.e2e.spec.ts Outdated
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

@frederickobrien frederickobrien added the run_chromatic Runs chromatic when label is applied label Sep 1, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Sep 1, 2026
@frederickobrien
frederickobrien force-pushed the revamped-immersives-everywhere branch from 67412b8 to 3401971 Compare September 1, 2026 11:04
@frederickobrien frederickobrien added the run_chromatic Runs chromatic when label is applied label Sep 1, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Sep 1, 2026

@emma-imber emma-imber left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sensible approach!

@frederickobrien
frederickobrien force-pushed the revamped-immersives-everywhere branch from d9073d8 to fc904a2 Compare September 1, 2026 19:26
@frederickobrien frederickobrien added the run_chromatic Runs chromatic when label is applied label Sep 1, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Sep 1, 2026
@frederickobrien
frederickobrien added this pull request to the merge queue Sep 2, 2026
Merged via the queue into main with commit 8bdaf92 Sep 2, 2026
41 checks passed
@frederickobrien
frederickobrien deleted the revamped-immersives-everywhere branch September 2, 2026 09:50
@gu-prout

gu-prout Bot commented Sep 2, 2026

Copy link
Copy Markdown

Seen on PROD (merged by @frederickobrien 8 minutes and 44 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature Seen-on-PROD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants