Skip to content

Commit c0d0e21

Browse files
committed
remove gallery specific series link styling
1 parent 21a9353 commit c0d0e21

1 file changed

Lines changed: 18 additions & 40 deletions

File tree

dotcom-rendering/src/components/SeriesSectionLink.tsx

Lines changed: 18 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import { css } from '@emotion/react';
22
import {
3-
between,
43
from,
54
headlineBold17,
65
headlineBold20,
@@ -83,23 +82,14 @@ const marginRight = css`
8382
}
8483
`;
8584

86-
const invertedStyle = (design: ArticleDesign) => {
87-
if (design === ArticleDesign.Gallery) {
88-
return '';
89-
}
90-
return css`
91-
/* Handle text wrapping onto a new line */
92-
white-space: pre-wrap;
93-
box-decoration-break: clone;
94-
line-height: 28px;
95-
${from.leftCol} {
96-
line-height: 28px;
97-
}
98-
padding-right: ${space[1]}px;
99-
padding-top: ${space[1]}px;
100-
padding-bottom: ${space[3]}px;
101-
`;
102-
};
85+
const invertedStyle = css`
86+
/* Handle text wrapping onto a new line */
87+
white-space: pre-wrap;
88+
box-decoration-break: clone;
89+
padding-right: ${space[2]}px;
90+
padding-top: ${space[1]}px;
91+
padding-bottom: ${space[2]}px;
92+
`;
10393

10494
const fontStyles = (format: ArticleFormat) => {
10595
switch (format.design) {
@@ -184,26 +174,15 @@ const breakWord = css`
184174
word-break: break-word;
185175
`;
186176

187-
const sectionPadding = (design: ArticleDesign) => {
188-
if (design === ArticleDesign.Gallery) {
189-
return css`
190-
padding: 0 ${space[2]}px 0 ${space[3]}px;
191-
192-
${between.mobileLandscape.and.tablet} {
193-
padding-left: ${space[5]}px;
194-
}
195-
`;
177+
const sectionPadding = css`
178+
padding-left: 10px;
179+
${from.mobileLandscape} {
180+
padding-left: 18px;
196181
}
197-
return css`
198-
padding-left: 10px;
199-
${from.mobileLandscape} {
200-
padding-left: 18px;
201-
}
202-
${from.tablet} {
203-
padding-left: ${space[1]}px;
204-
}
205-
`;
206-
};
182+
${from.tablet} {
183+
padding-left: ${space[3]}px;
184+
}
185+
`;
207186

208187
export const SeriesSectionLink = ({
209188
format,
@@ -358,9 +337,9 @@ export const SeriesSectionLink = ({
358337
css={[
359338
sectionLabelLink,
360339
fontStyles(format),
361-
invertedStyle(format.design),
340+
invertedStyle,
362341
breakWord,
363-
sectionPadding(format.design),
342+
sectionPadding,
364343
css`
365344
color: ${titleColour};
366345
background-color: ${themePalette(
@@ -369,7 +348,6 @@ export const SeriesSectionLink = ({
369348
`,
370349
format.design === ArticleDesign.Gallery &&
371350
css`
372-
display: inline-block;
373351
position: relative;
374352
`,
375353
format.display === ArticleDisplay.Immersive &&

0 commit comments

Comments
 (0)