From ada933b8ba1ee71270d767f52608e75a7b2adb2a Mon Sep 17 00:00:00 2001 From: Isabella Mitchell Date: Wed, 20 Mar 2024 10:50:47 +0000 Subject: [PATCH 1/4] WSTEAM1-847: Allow default paddingBottom on larger screen sizes --- ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx b/ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx index 5223c63cafe..9e5e959df7d 100644 --- a/ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx +++ b/ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx @@ -109,7 +109,8 @@ export default { paddingRight: `${spacings.DOUBLE}rem`, }, [mq.GROUP_4_MIN_WIDTH]: { - padding: 0, + paddingLeft: 0, + paddingRight: 0, }, }), }; From 4c7d38f8b46845154594fe0ff796aeeec2618c39 Mon Sep 17 00:00:00 2001 From: Isabella Mitchell Date: Wed, 20 Mar 2024 10:51:15 +0000 Subject: [PATCH 2/4] WSTEAM1-847: Add default paddingBottom for MediaLoader --- src/app/components/MediaLoader/index.styles.ts | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/app/components/MediaLoader/index.styles.ts b/src/app/components/MediaLoader/index.styles.ts index c1e830a739a..8aceca5a60f 100644 --- a/src/app/components/MediaLoader/index.styles.ts +++ b/src/app/components/MediaLoader/index.styles.ts @@ -1,9 +1,12 @@ import { css, Theme } from '@emotion/react'; export default { - figure: css({ - margin: 0, - }), + figure: ({ spacings }: Theme) => + css({ + margin: 0, + paddingBottom: `${spacings.TRIPLE}rem`, + width: '100%', + }), mediaContainer: ({ palette }: Theme) => css({ backgroundColor: palette.BLACK, From 8c911aa8c87f1c90e401d02fe01ecfc3bd3938ea Mon Sep 17 00:00:00 2001 From: Isabella Mitchell Date: Wed, 20 Mar 2024 11:32:13 +0000 Subject: [PATCH 3/4] wSTEAM1-837: Reduce bottom padding for live pages to align with designs --- ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx b/ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx index 9e5e959df7d..02f7d085503 100644 --- a/ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx +++ b/ws-nextjs-app/pages/[service]/live/[id]/Post/styles.tsx @@ -102,8 +102,7 @@ export default { }), bodyMedia: ({ spacings, mq }: Theme) => css({ - paddingLeft: `${spacings.FULL}rem`, - paddingRight: `${spacings.FULL}rem`, + padding: `0 ${spacings.FULL}rem ${spacings.DOUBLE}rem`, [mq.GROUP_2_MIN_WIDTH]: { paddingLeft: `${spacings.DOUBLE}rem`, paddingRight: `${spacings.DOUBLE}rem`, From b7c3a672f44a067a6b09e502ffcbbd0b2a48abfc Mon Sep 17 00:00:00 2001 From: Isabella Mitchell Date: Wed, 20 Mar 2024 11:57:42 +0000 Subject: [PATCH 4/4] WSTEAM1-847: Update snapshot --- .../pages/live/pidgin/__snapshots__/canonical.test.ts.snap | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ws-nextjs-app/integration/pages/live/pidgin/__snapshots__/canonical.test.ts.snap b/ws-nextjs-app/integration/pages/live/pidgin/__snapshots__/canonical.test.ts.snap index 3606fa0116a..62ae25d40ed 100644 --- a/ws-nextjs-app/integration/pages/live/pidgin/__snapshots__/canonical.test.ts.snap +++ b/ws-nextjs-app/integration/pages/live/pidgin/__snapshots__/canonical.test.ts.snap @@ -330,7 +330,7 @@ exports[`Canonical Live Media Loader renders a placeholder 1`] = ` exports[`Canonical Live Media Loader renders a valid container 1`] = `