fix padding

hashicorp · Sep 12, 2024 · 9cb20f6 · 9cb20f6
1 parent 50a82a5
commit 9cb20f6
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 4 deletions.
diff --git a/src/layouts/sidebar-sidecar/index.tsx b/src/layouts/sidebar-sidecar/index.tsx
@@ -93,14 +93,24 @@ const SidebarSidecarLayoutContent = ({
 		sidebarContent = <Sidebar {...sidebarProps} />
 	}
 
+	const shouldNotHaveSidePadding =
+		sidebarContent.props.title === 'Main Menu' ||
+		(AlternateSidebar && !sidebarProps?.menuItems)
+
 	return (
 		<div className={classNames(s.root, s[`mainWidth-${mainWidth}`])}>
 			<MobileMenuContainer className={s.sidebarContainer} ref={sidebarRef}>
 				<div className={s.sidebarContentWrapper}>
 					<MobileAuthenticationControls
-						className={s.mobileAuthControlsContainer}
+						className={classNames(s.mobileAuthControlsContainer, s.sidePadding)}
 					/>
-					{sidebarContent}
+					<div
+						className={classNames({
+							[s.sidePadding]: !shouldNotHaveSidePadding,
+						})}
+					>
+						{sidebarContent}
+					</div>
 				</div>
 			</MobileMenuContainer>
 			<div className={s.contentWrapper} ref={contentRef}>

diff --git a/src/layouts/sidebar-sidecar/sidebar-sidecar-layout.module.css b/src/layouts/sidebar-sidecar/sidebar-sidecar-layout.module.css
@@ -62,8 +62,6 @@ for further details.
 	flex-grow: 1;
 	overflow-y: auto;
 	position: relative;
-	padding-left: 16px;
-	padding-right: 16px;
 
 	@media (--dev-dot-hide-mobile-menu) {
 		padding-top: 24px;
@@ -197,3 +195,8 @@ for further details.
 .editOnGithubLink {
 	margin-top: 64px;
 }
+
+.sidePadding {
+	padding-left: 16px;
+	padding-right: 16px;
+}