From 77be5e5dcacca98382f00ed38f184f76f49a399b Mon Sep 17 00:00:00 2001 From: Jake Lee Kennedy <1731150+Jakeii@users.noreply.github.com> Date: Wed, 8 Jul 2026 17:16:54 +0100 Subject: [PATCH 1/4] release redesigned cricket header --- ab-testing/config/abTests.ts | 11 ------- dotcom-rendering/src/layouts/LiveLayout.tsx | 32 ++++--------------- .../src/layouts/StandardLayout.tsx | 10 +----- 3 files changed, 7 insertions(+), 46 deletions(-) diff --git a/ab-testing/config/abTests.ts b/ab-testing/config/abTests.ts index d3414585345..da0fc01626f 100644 --- a/ab-testing/config/abTests.ts +++ b/ab-testing/config/abTests.ts @@ -183,17 +183,6 @@ const ABTests: ABTest[] = [ groups: ["control", "variant-1"], shouldForceMetricsCollection: false, }, - { - name: "webx-cricket-redesign", - description: "Redesign of the cricket header and scorecard on web", - owners: ["dotcom.platform@theguardian.com"], - status: "ON", - expirationDate: "2026-08-01", - type: "server", - audienceSize: 0 / 100, - groups: ["enable"], - shouldForceMetricsCollection: false, - }, ]; const activeABtests = ABTests.filter((test) => test.status === "ON"); diff --git a/dotcom-rendering/src/layouts/LiveLayout.tsx b/dotcom-rendering/src/layouts/LiveLayout.tsx index 5887c0c2496..3b6a02b228b 100644 --- a/dotcom-rendering/src/layouts/LiveLayout.tsx +++ b/dotcom-rendering/src/layouts/LiveLayout.tsx @@ -27,7 +27,6 @@ import { DiscussionLayout } from '../components/DiscussionLayout'; import { FootballMatchHeaderWrapper } from '../components/FootballMatchHeaderWrapper.island'; import { FootballMiniMatchStatsWrapper } from '../components/FootballMiniMatchStatsWrapper.island'; import { Footer } from '../components/Footer'; -import { GetCricketScoreboard } from '../components/GetCricketScoreboard.island'; import { GridItem } from '../components/GridItem'; import { HeaderAdSlot } from '../components/HeaderAdSlot'; import { Island } from '../components/Island'; @@ -271,9 +270,6 @@ export const LiveLayout = (props: WebProps | AppsProps) => { } = article; const ab = useAB(); - const isCricketRedesignEnabled = Boolean( - ab?.isUserInTestGroup('webx-cricket-redesign', 'enable'), - ); // TODO: // 1) Read 'forceEpic' value from URL parameter and use it to force the slot to render @@ -395,7 +391,6 @@ export const LiveLayout = (props: WebProps | AppsProps) => { format={format} article={article} liveBlogAreaId={liveBlogAreaId} - isCricketRedesignEnabled={isCricketRedesignEnabled} /> {/* This element is used to replace the liveblog with the scorecard when the scorecard tab is clicked */} @@ -598,20 +593,6 @@ export const LiveLayout = (props: WebProps | AppsProps) => {
- {!!cricketMatchUrl && - !isCricketRedesignEnabled && ( - - - - )} { cricketMatchStatsUrl={ cricketMatchStatsUrl } - isCricketRedesignEnabled={ - isCricketRedesignEnabled - } + renderingTarget={renderingTarget} /> @@ -1139,8 +1118,10 @@ export const LiveLayout = (props: WebProps | AppsProps) => { const MiniMatchStats = (props: { footballMatchStatsUrl: string | undefined; cricketMatchStatsUrl: string | undefined; - isCricketRedesignEnabled: boolean; + renderingTarget: RenderingTarget; }) => { + const isApps = props.renderingTarget === 'Apps'; + if (props.footballMatchStatsUrl) { return ( @@ -1151,7 +1132,7 @@ const MiniMatchStats = (props: { ); } - if (props.cricketMatchStatsUrl && props.isCricketRedesignEnabled) { + if (!isApps && props.cricketMatchStatsUrl) { return ( { const footballMatchLeagueName = props.article.sectionLabel; const footballMatchLeagueUrl = `${props.article.guardianBaseURL}/${props.article.sectionUrl}`; @@ -1209,7 +1189,7 @@ const Header = (props: { ); } - if (!isApps && cricketMatchHeaderUrl && props.isCricketRedesignEnabled) { + if (!isApps && cricketMatchHeaderUrl) { return ( <>