Merge branch 'main' into filter/utm-into-xcust

charleycampbell · web-flow · commit eb4aff85e6db · 2025-11-26T14:25:33.000Z
diff --git a/.github/workflows/ab-testing-checks.yml b/.github/workflows/ab-testing-checks.yml
@@ -28,7 +28,7 @@ jobs:
       FASTLY_AB_TESTING_CONFIG: ${{ secrets.FASTLY_AB_TESTING_CONFIG }}
       FASTLY_API_TOKEN: ${{ secrets.FASTLY_API_TOKEN }}
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/ab-testing-deploy.yml b/.github/workflows/ab-testing-deploy.yml
@@ -24,7 +24,7 @@ jobs:
       run:
         working-directory: ab-testing/config
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/ab-testing-ui.yml b/.github/workflows/ab-testing-ui.yml
@@ -21,7 +21,7 @@ jobs:
     permissions:
       contents: read
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
@@ -47,7 +47,7 @@ jobs:
 
     steps:
       - name: Checkout
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           fetch-depth: 0
           persist-credentials: false
diff --git a/.github/workflows/ar-chromatic.yml b/.github/workflows/ar-chromatic.yml
@@ -19,7 +19,7 @@ jobs:
       github.event.pull_request.user.login != 'dependabot')
     steps:
       - name: Checkout - On Pull Request
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         if: ${{ github.event_name == 'pull_request'}}
         with:
           fetch-depth: 0
@@ -28,7 +28,7 @@ jobs:
           # Hopefully by checking out the HEAD commit of a PR instead of the merge commit we can avoid some of those issues.
           ref: ${{ github.event.pull_request.head.sha }}
       - name: Checkout - On Push Event
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         if: ${{ github.event_name == 'push'}}
         with:
           fetch-depth: 0
diff --git a/.github/workflows/ar-ci.yml b/.github/workflows/ar-ci.yml
@@ -23,7 +23,7 @@ jobs:
 
     # Checkout the repository
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/ar-nodejs.yml b/.github/workflows/ar-nodejs.yml
@@ -18,7 +18,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/build-check.yml b/.github/workflows/build-check.yml
@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/bundle-analyser.yml b/.github/workflows/bundle-analyser.yml
@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/container.yml b/.github/workflows/container.yml
@@ -19,7 +19,7 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/dcr-chromatic.yml b/.github/workflows/dcr-chromatic.yml
@@ -15,7 +15,7 @@ jobs:
     if: contains(github.event.pull_request.labels.*.name, 'run_chromatic')  ||  github.ref == 'refs/heads/main'
     steps:
       - name: Checkout - On Pull Request
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         if: ${{ github.event_name == 'pull_request'}}
         with:
           fetch-depth: 0
@@ -24,7 +24,7 @@ jobs:
           # Hopefully by checking out the HEAD commit of a PR instead of the merge commit we can avoid some of those issues.
           ref: ${{ github.event.pull_request.head.sha }}
       - name: Checkout - On Push Event
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         if: ${{ github.event_name == 'push'}}
         with:
           fetch-depth: 0
diff --git a/.github/workflows/deno.yml b/.github/workflows/deno.yml
@@ -9,7 +9,7 @@ jobs:
     name: 🦕 Deno health
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       # https://github.com/denoland/setup-deno#latest-stable-for-a-major
       - uses: denoland/setup-deno@v1
diff --git a/.github/workflows/jest.yml b/.github/workflows/jest.yml
@@ -5,7 +5,7 @@ jobs:
   jest:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/lighthouse.yml b/.github/workflows/lighthouse.yml
@@ -14,7 +14,7 @@ jobs:
           - 'http://localhost:9000/Front/https://www.theguardian.com/uk'
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -5,7 +5,7 @@ jobs:
   check:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/playwright.yml b/.github/workflows/playwright.yml
@@ -21,7 +21,7 @@ jobs:
           - 9000:9000
     steps:
       - name: Checkout
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/prettier.yml b/.github/workflows/prettier.yml
@@ -5,7 +5,7 @@ jobs:
   check:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/scheduled.yml b/.github/workflows/scheduled.yml
@@ -16,7 +16,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       # https://github.com/denoland/setup-deno#latest-stable-for-a-major
       - uses: denoland/setup-deno@v1
diff --git a/.github/workflows/schema-check.yml b/.github/workflows/schema-check.yml
@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/stories-check.yml b/.github/workflows/stories-check.yml
@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/.github/workflows/typescript.yml b/.github/workflows/typescript.yml
@@ -10,7 +10,7 @@ jobs:
     name: Typescript
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - name: Set up Node environment
         uses: ./.github/actions/setup-node-env
diff --git a/ab-testing/config/abTests.ts b/ab-testing/config/abTests.ts
@@ -53,7 +53,7 @@ const ABTests: ABTest[] = [
 		status: "ON",
 		expirationDate: "2025-12-30",
 		type: "server",
-		audienceSize: 0 / 100,
+		audienceSize: 10 / 100,
 		groups: ["control", "variant"],
 		shouldForceMetricsCollection: false,
 	},
diff --git a/dotcom-rendering/package.json b/dotcom-rendering/package.json
@@ -102,7 +102,6 @@
 		"constructs": "10.4.2",
 		"cpy": "11.0.0",
 		"css-loader": "7.1.2",
-		"curlyquotes": "1.5.5",
 		"dompurify": "3.2.4",
 		"dynamic-import-polyfill": "0.1.1",
 		"eslint": "8.57.1",
diff --git a/dotcom-rendering/src/components/Card/Card.tsx b/dotcom-rendering/src/components/Card/Card.tsx
@@ -844,7 +844,6 @@ export const Card = ({
 			format={format}
 			showTopBarDesktop={showTopBarDesktop}
 			showTopBarMobile={showTopBarMobile}
-			isOnwardContent={isOnwardContent}
 			containerPalette={containerPalette}
 		>
 			<CardLink
@@ -1081,10 +1080,6 @@ export const Card = ({
 											imageSize={mediaSize}
 											alt={headlineText}
 											loading={imageLoading}
-											roundedCorners={
-												isOnwardContent &&
-												!isMoreGalleriesOnwardContent
-											}
 											aspectRatio={aspectRatio}
 										/>
 									</div>
@@ -1098,10 +1093,6 @@ export const Card = ({
 									imageSize={mediaSize}
 									alt={media.imageAltText}
 									loading={imageLoading}
-									roundedCorners={
-										isOnwardContent &&
-										!isMoreGalleriesOnwardContent
-									}
 									aspectRatio={aspectRatio}
 								/>
 								{isVideoMainMedia && mainMedia.duration > 0 && (
@@ -1143,10 +1134,6 @@ export const Card = ({
 											imageSize="small"
 											alt={media.imageAltText}
 											loading={imageLoading}
-											roundedCorners={
-												isOnwardContent &&
-												!isMoreGalleriesOnwardContent
-											}
 											aspectRatio="1:1"
 										/>
 									</div>
diff --git a/dotcom-rendering/src/components/Card/components/CardWrapper.tsx b/dotcom-rendering/src/components/Card/components/CardWrapper.tsx
@@ -1,6 +1,6 @@
 import { css } from '@emotion/react';
 import { from, space, until } from '@guardian/source/foundations';
-import { ArticleDesign, type ArticleFormat } from '../../../lib/articleFormat';
+import type { ArticleFormat } from '../../../lib/articleFormat';
 import { palette } from '../../../palette';
 import type { DCRContainerPalette } from '../../../types/front';
 import { ContainerOverrides } from '../../ContainerOverrides';
@@ -11,7 +11,6 @@ type Props = {
 	format: ArticleFormat;
 	showTopBarDesktop: boolean;
 	showTopBarMobile: boolean;
-	isOnwardContent: boolean;
 	containerPalette?: DCRContainerPalette;
 };
 
@@ -93,21 +92,11 @@ const desktopTopBarStyles = css`
 	}
 `;
 
-const onwardContentStyles = css`
-	border-radius: ${space[2]}px;
-	overflow: hidden;
-
-	:hover .media-overlay {
-		border-radius: ${space[2]}px;
-	}
-`;
-
 export const CardWrapper = ({
 	children,
 	format,
 	showTopBarDesktop,
 	showTopBarMobile,
-	isOnwardContent,
 	containerPalette,
 }: Props) => {
 	return (
@@ -119,9 +108,6 @@ export const CardWrapper = ({
 						hoverStyles,
 						showTopBarDesktop && desktopTopBarStyles,
 						showTopBarMobile && mobileTopBarStyles,
-						isOnwardContent &&
-							format.design !== ArticleDesign.Gallery &&
-							onwardContentStyles,
 					]}
 				>
 					{children}
diff --git a/dotcom-rendering/src/components/CardPicture.tsx b/dotcom-rendering/src/components/CardPicture.tsx
@@ -1,5 +1,5 @@
 import { css } from '@emotion/react';
-import { breakpoints, space, until } from '@guardian/source/foundations';
+import { breakpoints, until } from '@guardian/source/foundations';
 import type { ImgHTMLAttributes } from 'react';
 import React from 'react';
 import type { AspectRatio } from '../types/front';
@@ -14,7 +14,6 @@ export type Props = {
 	mainImage: string;
 	loading: Loading;
 	alt?: string;
-	roundedCorners?: boolean;
 	isCircular?: boolean;
 	aspectRatio?: AspectRatio;
 	mobileAspectRatio?: AspectRatio;
@@ -200,12 +199,6 @@ const decideAspectRatioStyles = (aspectRatio?: AspectRatio) => {
 	`;
 };
 
-const borderRadius = css`
-	& > * {
-		border-radius: ${space[2]}px;
-	}
-`;
-
 const circularStyles = css`
 	border-radius: 100%;
 	object-fit: cover;
@@ -227,7 +220,6 @@ export const CardPicture = ({
 	alt,
 	imageSize,
 	loading,
-	roundedCorners,
 	isCircular,
 	aspectRatio = '5:3',
 	mobileAspectRatio,
@@ -251,7 +243,6 @@ export const CardPicture = ({
 				decideAspectRatioStyles(aspectRatio),
 				mobileAspectRatio &&
 					decideMobileAspectRatioStyles(mobileAspectRatio),
-				roundedCorners && borderRadius,
 				isCircular && circularStyles,
 			]}
 		>
diff --git a/dotcom-rendering/src/components/ListenToArticle.importable.tsx b/dotcom-rendering/src/components/ListenToArticle.importable.tsx
diff --git a/dotcom-rendering/src/components/SignInGate/signInGateMappings.ts b/dotcom-rendering/src/components/SignInGate/signInGateMappings.ts
diff --git a/dotcom-rendering/src/experiments/ab-tests.ts b/dotcom-rendering/src/experiments/ab-tests.ts
diff --git a/dotcom-rendering/src/experiments/tests/auxia-sign-in-gate.ts b/dotcom-rendering/src/experiments/tests/auxia-sign-in-gate.ts
diff --git a/dotcom-rendering/src/experiments/tests/sign-in-gate-main-control.ts b/dotcom-rendering/src/experiments/tests/sign-in-gate-main-control.ts
diff --git a/dotcom-rendering/src/experiments/tests/sign-in-gate-main-variant.ts b/dotcom-rendering/src/experiments/tests/sign-in-gate-main-variant.ts
diff --git a/dotcom-rendering/src/lib/useSignInGateSelector.ts b/dotcom-rendering/src/lib/useSignInGateSelector.ts
diff --git a/dotcom-rendering/src/lib/useSignInGateWillShow.ts b/dotcom-rendering/src/lib/useSignInGateWillShow.ts
diff --git a/dotcom-rendering/src/paletteDeclarations.ts b/dotcom-rendering/src/paletteDeclarations.ts
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
