Skip to content

Commit 4dbb372

Browse files
committed
Change to lighter grey, start background after ad label and add it to just variant
1 parent 0f7f969 commit 4dbb372

File tree

1 file changed

+12
-15
lines changed

1 file changed

+12
-15
lines changed

dotcom-rendering/src/components/AdSlot.web.tsx

Lines changed: 12 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ const topAboveNavContainerStyles = css`
115115
display: block;
116116
`;
117117

118-
const topAboveNavContainerVaraintStyles = css`
118+
const topAboveNavContainerVariantStyles = css`
119119
padding-bottom: ${space[5]}px;
120120
position: relative;
121121
margin: 0 auto;
@@ -128,6 +128,13 @@ const topAboveNavContainerVaraintStyles = css`
128128
&[top-above-nav-ad-rendered='true'] {
129129
min-height: auto;
130130
}
131+
132+
.ad-slot--top-above-nav:not([data-google-query-id]) {
133+
margin: 24px auto 0;
134+
height: 250px;
135+
width: 970px;
136+
background-color: ${palette.neutral[93]};
137+
}
131138
`;
132139

133140
/**
@@ -405,15 +412,6 @@ const crosswordBannerMobileAdStyles = css`
405412
min-height: ${getMinHeight(adSizes.mobilesticky.height)}px;
406413
`;
407414

408-
const topAboveNavBackground = css`
409-
.ad-slot--top-above-nav:not([data-google-query-id]) {
410-
margin: 0 auto;
411-
height: ${getMinHeight(250)}px;
412-
width: 970px;
413-
background-color: ${palette.neutral[86]};
414-
}
415-
`;
416-
417415
const AdSlotWrapper = ({
418416
children,
419417
css: additionalCss,
@@ -607,12 +605,11 @@ export const AdSlot = ({
607605
case 'top-above-nav': {
608606
return (
609607
<AdSlotWrapper
610-
css={[
608+
css={
611609
isIn250ReservationVariant
612-
? topAboveNavContainerVaraintStyles
613-
: topAboveNavContainerStyles,
614-
topAboveNavBackground,
615-
]}
610+
? topAboveNavContainerVariantStyles
611+
: topAboveNavContainerStyles
612+
}
616613
>
617614
<div
618615
id="dfp-ad--top-above-nav"

0 commit comments

Comments
 (0)