From c6d40dfc46e9092c615c40177dcc3eed278a65f3 Mon Sep 17 00:00:00 2001
From: Charlotte <charlotte.emms@theguardian.com>
Date: Fri, 29 Nov 2024 17:43:41 +0000
Subject: [PATCH] remove container overrides wrapper for FrontSection and
 LabsSection components

since these components have container overrides inside them so are currently being duplicated unnecessarily
---
 dotcom-rendering/src/layouts/FrontLayout.tsx | 28 ++++++--------------
 1 file changed, 8 insertions(+), 20 deletions(-)

diff --git a/dotcom-rendering/src/layouts/FrontLayout.tsx b/dotcom-rendering/src/layouts/FrontLayout.tsx
index 58db63a1e5..c2a5160946 100644
--- a/dotcom-rendering/src/layouts/FrontLayout.tsx
+++ b/dotcom-rendering/src/layouts/FrontLayout.tsx
@@ -430,10 +430,7 @@ export const FrontLayout = ({ front, NAV }: Props) => {
 							: undefined;
 
 						return (
-							<ContainerOverrides
-								key={ophanName}
-								containerPalette={collection.containerPalette}
-							>
+							<div key={ophanName}>
 								{decideFrontsBannerAdSlot(
 									renderAds,
 									hasPageSkin,
@@ -493,16 +490,13 @@ export const FrontLayout = ({ front, NAV }: Props) => {
 									mobileAdPositions,
 									hasPageSkin,
 								)}
-							</ContainerOverrides>
+							</div>
 						);
 					}
 
 					if (collection.containerPalette === 'Branded') {
 						return (
-							<ContainerOverrides
-								key={ophanName}
-								containerPalette={collection.containerPalette}
-							>
+							<div key={ophanName}>
 								<LabsSection
 									title={collection.displayName}
 									collectionId={collection.id}
@@ -548,7 +542,7 @@ export const FrontLayout = ({ front, NAV }: Props) => {
 									mobileAdPositions,
 									hasPageSkin,
 								)}
-							</ContainerOverrides>
+							</div>
 						);
 					}
 
@@ -560,10 +554,7 @@ export const FrontLayout = ({ front, NAV }: Props) => {
 							collection.containerPalette ?? 'MediaPalette';
 
 						return (
-							<ContainerOverrides
-								key={ophanName}
-								containerPalette={containerPalette}
-							>
+							<div key={ophanName}>
 								{decideFrontsBannerAdSlot(
 									renderAds,
 									hasPageSkin,
@@ -635,15 +626,12 @@ export const FrontLayout = ({ front, NAV }: Props) => {
 										mobileAdPositions,
 										hasPageSkin,
 									)}
-							</ContainerOverrides>
+							</div>
 						);
 					}
 
 					return (
-						<ContainerOverrides
-							key={ophanName}
-							containerPalette={collection.containerPalette}
-						>
+						<div key={ophanName}>
 							{decideFrontsBannerAdSlot(
 								renderAds,
 								hasPageSkin,
@@ -718,7 +706,7 @@ export const FrontLayout = ({ front, NAV }: Props) => {
 								mobileAdPositions,
 								hasPageSkin,
 							)}
-						</ContainerOverrides>
+						</div>
 					);
 				})}
 			</main>