Skip to content

Commit 1448372

Browse files
deedeehmarjisound
andcommitted
Update headline and branding styling for Labs Galleries
Co-authored-by: Marjan Kalanaki <[email protected]>
1 parent ca06088 commit 1448372

File tree

2 files changed

+37
-22
lines changed

2 files changed

+37
-22
lines changed

dotcom-rendering/src/components/ArticleHeadline.tsx

Lines changed: 32 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -143,14 +143,27 @@ const invertedFontLineHeight = css`
143143
}
144144
`;
145145

146-
const labsFont = css`
147-
${textSansBold28};
148-
line-height: 2rem;
149-
${from.tablet} {
150-
${textSansBold34};
151-
line-height: 2.375rem;
146+
const labsFont = (design: ArticleDesign) => {
147+
if (design === ArticleDesign.Gallery) {
148+
return css`
149+
${textSansBold34};
150+
line-height: 2.1875rem;
151+
${from.desktop} {
152+
${textSansBold34};
153+
font-size: 50px;
154+
line-height: 3.125rem;
155+
}
156+
`;
152157
}
153-
`;
158+
return css`
159+
${textSansBold28};
160+
line-height: 2rem;
161+
${from.tablet} {
162+
${textSansBold34};
163+
line-height: 2.375rem;
164+
}
165+
`;
166+
};
154167

155168
const jumboLabsFont = css`
156169
${textSansBold34};
@@ -426,7 +439,7 @@ export const ArticleHeadline = ({
426439
<h1
427440
css={[
428441
format.theme === ArticleSpecial.Labs
429-
? labsFont
442+
? labsFont(format.design)
430443
: headlineFont(format),
431444
invertedText,
432445
css`
@@ -508,7 +521,7 @@ export const ArticleHeadline = ({
508521
<h1
509522
css={[
510523
format.theme === ArticleSpecial.Labs
511-
? labsFont
524+
? labsFont(format.design)
512525
: headlineFont(format),
513526
topPadding,
514527
css`
@@ -549,7 +562,7 @@ export const ArticleHeadline = ({
549562
<h1
550563
css={[
551564
format.theme === ArticleSpecial.Labs
552-
? labsFont
565+
? labsFont(format.design)
553566
: headlineFont(format),
554567
topPadding,
555568
css`
@@ -584,7 +597,7 @@ export const ArticleHeadline = ({
584597
<h1
585598
css={[
586599
format.theme === ArticleSpecial.Labs
587-
? labsFont
600+
? labsFont(format.design)
588601
: headlineFont(format),
589602
topPadding,
590603
css`
@@ -626,7 +639,7 @@ export const ArticleHeadline = ({
626639
<h1
627640
css={[
628641
format.theme === ArticleSpecial.Labs
629-
? labsFont
642+
? labsFont(format.design)
630643
: headlineFont(format),
631644
topPadding,
632645
css`
@@ -667,7 +680,7 @@ export const ArticleHeadline = ({
667680
<h1
668681
css={[
669682
format.theme === ArticleSpecial.Labs
670-
? labsFont
683+
? labsFont(format.design)
671684
: headlineFont(format),
672685
invertedWrapper,
673686
invertedFontLineHeight,
@@ -718,7 +731,7 @@ export const ArticleHeadline = ({
718731
<h1
719732
css={[
720733
format.theme === ArticleSpecial.Labs
721-
? labsFont
734+
? labsFont(format.design)
722735
: headlineFont(format),
723736
topPadding,
724737
css`
@@ -760,7 +773,7 @@ export const ArticleHeadline = ({
760773
<h1
761774
css={[
762775
format.theme === ArticleSpecial.Labs
763-
? labsFont
776+
? labsFont(format.design)
764777
: headlineFont(format),
765778
css`
766779
color: ${isMatch
@@ -805,7 +818,7 @@ export const ArticleHeadline = ({
805818
}
806819
css={[
807820
format.theme === ArticleSpecial.Labs
808-
? labsFont
821+
? labsFont(format.design)
809822
: headlineFont(format),
810823
topPadding,
811824
css`
@@ -832,7 +845,7 @@ export const ArticleHeadline = ({
832845
<h1
833846
css={[
834847
format.theme === ArticleSpecial.Labs
835-
? labsFont
848+
? labsFont(format.design)
836849
: headlineFont(format),
837850
topPadding,
838851
css`
@@ -859,7 +872,7 @@ export const ArticleHeadline = ({
859872
<h1
860873
css={[
861874
format.theme === ArticleSpecial.Labs
862-
? labsFont
875+
? labsFont(format.design)
863876
: headlineFont(format),
864877
css`
865878
color: ${themePalette(
@@ -905,7 +918,7 @@ export const ArticleHeadline = ({
905918
<h1
906919
css={[
907920
format.theme === ArticleSpecial.Labs
908-
? labsFont
921+
? labsFont(format.design)
909922
: headlineFont(format),
910923
topPadding,
911924
css`

dotcom-rendering/src/paletteDeclarations.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2040,7 +2040,7 @@ const brandingLinkDark: PaletteFunction = ({ design, theme }) => {
20402040
case Pillar.Lifestyle:
20412041
return pillarPalette(theme, 500);
20422042
case ArticleSpecial.Labs:
2043-
return sourcePalette.specialReport[500];
2043+
return sourcePalette.neutral[73];
20442044
case ArticleSpecial.SpecialReport:
20452045
return sourcePalette.specialReport[500];
20462046
case ArticleSpecial.SpecialReportAlt:
@@ -3854,6 +3854,9 @@ const liveBlockBorderBottomDark: PaletteFunction = () =>
38543854
const subMetaLabelTextLight: PaletteFunction = ({ theme, design }) => {
38553855
switch (theme) {
38563856
case ArticleSpecial.Labs:
3857+
if (design === ArticleDesign.Gallery) {
3858+
return sourcePalette.neutral[60];
3859+
}
38573860
return sourcePalette.neutral[7];
38583861
case ArticleSpecial.SpecialReport:
38593862
return sourcePalette.specialReport[300];
@@ -3870,9 +3873,8 @@ const subMetaLabelTextLight: PaletteFunction = ({ theme, design }) => {
38703873
case ArticleDesign.Picture:
38713874
case ArticleDesign.Video:
38723875
case ArticleDesign.Audio:
3873-
return sourcePalette.neutral[60];
38743876
case ArticleDesign.Gallery:
3875-
return sourcePalette.neutral[73];
3877+
return sourcePalette.neutral[60];
38763878
default:
38773879
return sourcePalette.neutral[46];
38783880
}

0 commit comments

Comments
 (0)