Skip to content

Commit 5cf6c69

Browse files
authored
Merge pull request #14439 from guardian/doml/highlights-small-screens
Update font sizes at mobile breakpoints
2 parents d6d0fae + b51b6b8 commit 5cf6c69

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

dotcom-rendering/src/components/CardHeadline.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ export const CardHeadline = ({
227227
css={css`
228228
color: ${headlineColour};
229229
`}
230-
className="show-underline"
230+
className="show-underline headline-text"
231231
>
232232
{headlineText}
233233
{isExternalLink && (

dotcom-rendering/src/components/Masthead/HighlightsCard.tsx

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,6 +100,16 @@ const content = css`
100100
${from.tablet} {
101101
padding-bottom: 10px;
102102
}
103+
104+
/**
105+
* We're deliberately using a font-size that is not in Source so that
106+
* the headline doesn't exceed three lines across all mobile breakpoints
107+
*/
108+
${between.mobileMedium.and.mobileLandscape} {
109+
.headline-text {
110+
font-size: 1rem;
111+
}
112+
}
103113
`;
104114

105115
const starWrapper = css`
@@ -144,7 +154,7 @@ export const HighlightsCard = ({
144154
desktop: 'xxsmall',
145155
tablet: 'xxsmall',
146156
mobileMedium: 'xxsmall',
147-
mobile: 'tiny',
157+
mobile: 'xxxsmall',
148158
}}
149159
showPulsingDot={
150160
format.design === ArticleDesign.LiveBlog

dotcom-rendering/src/components/ScrollableHighlights.stories.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ const meta: Meta<typeof ScrollableHighlights> = {
1111
chromatic: {
1212
viewports: [
1313
breakpoints.mobile,
14+
breakpoints.mobileMedium,
1415
breakpoints.mobileLandscape,
1516
breakpoints.tablet,
1617
breakpoints.wide,

0 commit comments

Comments
 (0)