From 788e44f0dc55352f2ac4754e7b2a1c1e6529b6b9 Mon Sep 17 00:00:00 2001 From: Jon Mulhern <81927768+curlyfriesplease@users.noreply.github.com> Date: Fri, 15 May 2026 17:51:58 +0100 Subject: [PATCH 1/4] Removes margin from the accordion's chevron --- src/components/Molecules/Accordion/Accordion.js | 1 + .../Molecules/Accordion/__snapshots__/Accordion.test.js.snap | 1 + 2 files changed, 2 insertions(+) diff --git a/src/components/Molecules/Accordion/Accordion.js b/src/components/Molecules/Accordion/Accordion.js index ab3a3cddb..b30dcf0eb 100644 --- a/src/components/Molecules/Accordion/Accordion.js +++ b/src/components/Molecules/Accordion/Accordion.js @@ -54,6 +54,7 @@ const MenuGroupIcon = styled.img` height: 24px; transform: ${({ isOpen }) => (isOpen ? 'rotate(180deg)' : 'rotate(0deg)')}; transition: transform 0.15s ease-in-out; + margin: 0; `; const Copy = styled.div` diff --git a/src/components/Molecules/Accordion/__snapshots__/Accordion.test.js.snap b/src/components/Molecules/Accordion/__snapshots__/Accordion.test.js.snap index 8dbf90eb7..d7975400e 100644 --- a/src/components/Molecules/Accordion/__snapshots__/Accordion.test.js.snap +++ b/src/components/Molecules/Accordion/__snapshots__/Accordion.test.js.snap @@ -125,6 +125,7 @@ exports[`renders correctly 1`] = ` -webkit-transition: -webkit-transform 0.15s ease-in-out; -webkit-transition: transform 0.15s ease-in-out; transition: transform 0.15s ease-in-out; + margin: 0; } .c7 { From 1479fc4b7e01dd106c612d7ac92f5068733a97a3 Mon Sep 17 00:00:00 2001 From: Jon Mulhern <81927768+curlyfriesplease@users.noreply.github.com> Date: Fri, 15 May 2026 17:55:44 +0100 Subject: [PATCH 2/4] fix: Empty commit to please GA rule From 3773797951d2ec1abdd84176bb0f87c4f211b655 Mon Sep 17 00:00:00 2001 From: Jon Mulhern <81927768+curlyfriesplease@users.noreply.github.com> Date: Mon, 18 May 2026 11:41:19 +0100 Subject: [PATCH 3/4] Increase specificity to avoid being overridden by the .InnerCopy class which adds a bottom margin --- .../Molecules/Accordion/Accordion.js | 23 +++++++++++-------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/src/components/Molecules/Accordion/Accordion.js b/src/components/Molecules/Accordion/Accordion.js index b30dcf0eb..a2f89bbee 100644 --- a/src/components/Molecules/Accordion/Accordion.js +++ b/src/components/Molecules/Accordion/Accordion.js @@ -7,15 +7,6 @@ import spacing from '../../../theme/shared/spacing'; import menuGroupIcon from '../../../theme/shared/assets/Menu-Group-Icon.svg'; import defaultBoxShadow from '../../../theme/shared/boxShadows'; -const Container = styled.div` - border-radius: 1rem; - background: ${({ theme }) => theme.color('white')}; - ${defaultBoxShadow()} - &:hover { - ${defaultBoxShadow(true)} - } -`; - const ChevronKeyframes = keyframes` 0% { margin-top: 0rem; } 50% { margin-top: 0.5rem; } @@ -54,7 +45,19 @@ const MenuGroupIcon = styled.img` height: 24px; transform: ${({ isOpen }) => (isOpen ? 'rotate(180deg)' : 'rotate(0deg)')}; transition: transform 0.15s ease-in-out; - margin: 0; +`; + +const Container = styled.div` + border-radius: 1rem; + background: ${({ theme }) => theme.color('white')}; + ${defaultBoxShadow()} + &:hover { + ${defaultBoxShadow(true)} + } + + ${MenuGroupIcon} { + margin: 0; + } `; const Copy = styled.div` From fc3248e1fdb5f4c3fd6b9949112f5589e3368cbd Mon Sep 17 00:00:00 2001 From: Jon Mulhern <81927768+curlyfriesplease@users.noreply.github.com> Date: Mon, 18 May 2026 11:49:16 +0100 Subject: [PATCH 4/4] Snapshot --- .../__snapshots__/Accordion.test.js.snap | 45 ++++++++++--------- 1 file changed, 24 insertions(+), 21 deletions(-) diff --git a/src/components/Molecules/Accordion/__snapshots__/Accordion.test.js.snap b/src/components/Molecules/Accordion/__snapshots__/Accordion.test.js.snap index d7975400e..cf95463b4 100644 --- a/src/components/Molecules/Accordion/__snapshots__/Accordion.test.js.snap +++ b/src/components/Molecules/Accordion/__snapshots__/Accordion.test.js.snap @@ -40,7 +40,7 @@ exports[`renders correctly 1`] = ` line-height: inherit; } -.c8 { +.c9 { font-family: 'Montserrat',Helvetica,Arial,sans-serif; font-weight: 400; text-transform: inherit; @@ -52,26 +52,16 @@ exports[`renders correctly 1`] = ` line-height: 1.25rem; } -.c8 { +.c9 { font-size: 1rem; line-height: normal; } -.c8 span { +.c9 span { font-size: inherit; line-height: inherit; } -.c0 { - border-radius: 1rem; - background: #FFFFFF; - box-shadow: rgba(0,0,0,0.15) 0px 0px 1rem; -} - -.c0:hover { - box-shadow: rgba(0,0,0,0.25) 0px 0px 1rem; -} - .c1 { display: -webkit-box; display: -webkit-flex; @@ -117,7 +107,7 @@ exports[`renders correctly 1`] = ` align-content: center; } -.c6 { +.c7 { height: 24px; -webkit-transform: rotate(0deg); -ms-transform: rotate(0deg); @@ -125,10 +115,23 @@ exports[`renders correctly 1`] = ` -webkit-transition: -webkit-transform 0.15s ease-in-out; -webkit-transition: transform 0.15s ease-in-out; transition: transform 0.15s ease-in-out; +} + +.c0 { + border-radius: 1rem; + background: #FFFFFF; + box-shadow: rgba(0,0,0,0.15) 0px 0px 1rem; +} + +.c0:hover { + box-shadow: rgba(0,0,0,0.25) 0px 0px 1rem; +} + +.c0 .c6 { margin: 0; } -.c7 { +.c8 { overflow: hidden; height: 0; visibility: none; @@ -170,21 +173,21 @@ exports[`renders correctly 1`] = ` } @media (min-width:740px) { - .c8 { + .c9 { font-size: 1rem; line-height: 1.25rem; } } @media (min-width:1024px) { - .c8 { + .c9 { font-size: 1.125rem; line-height: 1.375rem; } } @media (min-width:740px) { - .c7 { + .c8 { padding: 0 3rem; } } @@ -213,16 +216,16 @@ exports[`renders correctly 1`] = `

Name, surname, email and billing address We need these to process your payment, create a receipt and send it to you. Establishment information We use this information to understand which institutions (e.g. schools, companies) raise money for us. Your details will be kept safe and never shared with other organisations; see our Privacy Policy for more information