Skip to content

Commit

Permalink
Display logo from tablet in choice cards banner
Browse files Browse the repository at this point in the history
Based on the designs, this logo should be visible in the tablet
breakpoint and not in the mobile medium breakpoint: looks like we made a
mistake with the breakpoint when we made this in the first place.

Interestingly, this is a change from some other banners: the guardian
weekly and print, for example. Those both display the logo from
mobileMedium. Not having seen the designs, I don’t know whether that’s
correct, but it has been that way for a long time so I assume so – I’ll
get this out and then check.
  • Loading branch information
emdash-ie authored and Emily Bourke committed Apr 4, 2023
1 parent 4cd9dc9 commit 3112be9
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ export const closeButtonStyles = css`

export const logoContainer = css`
display: none;
${from.mobileMedium} {
${from.tablet} {
display: block;
width: ${height.ctaSmall}px;
height: ${height.ctaSmall}px;
Expand Down

0 comments on commit 3112be9

Please sign in to comment.