From 3112be951a3902b5ba528ed1a47d78c9719ecf5b Mon Sep 17 00:00:00 2001
From: Emily Bourke <emily.bourke@guardian.co.uk>
Date: Tue, 4 Apr 2023 10:29:16 +0100
Subject: [PATCH] Display logo from tablet in choice cards banner
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

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.
---
 .../banners/choiceCardsBanner/choiceCardsBannerStyles.ts        | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/modules/src/modules/banners/choiceCardsBanner/choiceCardsBannerStyles.ts b/packages/modules/src/modules/banners/choiceCardsBanner/choiceCardsBannerStyles.ts
index 36f6f7dcd..507fe403f 100644
--- a/packages/modules/src/modules/banners/choiceCardsBanner/choiceCardsBannerStyles.ts
+++ b/packages/modules/src/modules/banners/choiceCardsBanner/choiceCardsBannerStyles.ts
@@ -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;