Skip to content

Commit

Permalink
Update labs styling
Browse files Browse the repository at this point in the history
  • Loading branch information
emma-imber committed Jan 8, 2025
1 parent 98f6570 commit 8fe7606
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 8 deletions.
13 changes: 7 additions & 6 deletions dotcom-rendering/src/components/AdSlot.web.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -143,12 +143,13 @@ const merchandisingAdStyles = css`
const rightAdStyles = css`
background-color: ${schemedPalette('--ad-background-article-inner')};
max-width: 300px;
`;

/**
* The right slot label should be dark on audio and video articles even
* in light mode. Other slots will stay the same as the right ad slot is
* the only one to overlay the dark section of Audio/Video pages.
*/
/*** The right slot label should be dark on audio and video articles even
* in light mode. Other slots will stay the same as the right ad slot is
* the only one to overlay the dark section of Audio/Video pages.
*/
const rightAdLabelStyles = css`
.ad-slot--right[data-label-show='true']::before {
background-color: ${schemedPalette('--ad-background-article-inner')};
border-top-color: ${schemedPalette('--ad-border-article-inner')};
Expand Down Expand Up @@ -453,7 +454,7 @@ export const AdSlot = ({
max-height: 100%;
`,
labelStyles,
rightAdStyles,
rightAdLabelStyles,
]}
>
<div
Expand Down
4 changes: 2 additions & 2 deletions dotcom-rendering/src/paletteDeclarations.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1915,7 +1915,7 @@ const articleInnerAdBackgroundLight: PaletteFunction = ({ design, theme }) => {
case ArticleDesign.Video:
switch (theme) {
case ArticleSpecial.Labs:
return sourcePalette.neutral[86];
return sourcePalette.neutral[97];
default:
return sourcePalette.neutral[0];
}
Expand All @@ -1942,7 +1942,7 @@ const articleInnerAdBorderLight: PaletteFunction = ({ design, theme }) => {
case ArticleDesign.Video:
switch (theme) {
case ArticleSpecial.Labs:
return sourcePalette.neutral[60];
return sourcePalette.neutral[86];
default:
return sourcePalette.neutral[20];
}
Expand Down

0 comments on commit 8fe7606

Please sign in to comment.