Move spacing to the controls container rather than on the items

guardian · Jan 21, 2025 · 761702e · 761702e
1 parent 27f944a
commit 761702e
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/dotcom-rendering/src/components/CarouselNavigationButtons.tsx b/dotcom-rendering/src/components/CarouselNavigationButtons.tsx
@@ -39,7 +39,6 @@ const buttonStyles = css`
 		display: flex;
 		gap: ${space[1]}px;
 		margin-left: auto;
-		padding-top: ${space[2]}px;
 	}
 `;
 

diff --git a/dotcom-rendering/src/components/FrontSection.tsx b/dotcom-rendering/src/components/FrontSection.tsx
@@ -282,7 +282,7 @@ const sectionControls = css`
 	grid-column: hide;
 	justify-self: end;
 	display: flex;
-	align-items: flex-end;
+	padding-top: ${space[2]}px;
 	${from.wide} {
 		flex-direction: column-reverse;
 		justify-content: flex-end;

diff --git a/dotcom-rendering/src/components/ShowHideButton.tsx b/dotcom-rendering/src/components/ShowHideButton.tsx
@@ -1,5 +1,5 @@
 import { css } from '@emotion/react';
-import { space, textSans14 } from '@guardian/source/foundations';
+import { textSans14 } from '@guardian/source/foundations';
 import { ButtonLink } from '@guardian/source/react-components';
 import { palette } from '../palette';
 
@@ -9,10 +9,7 @@ type Props = {
 
 const showHideButtonCss = css`
 	${textSans14};
-
-	margin-top: ${space[2]}px;
 	margin-right: 10px;
-	margin-bottom: ${space[2]}px;
 	position: relative;
 	align-items: bottom;
 	text-decoration: none;