Skip to content

Commit 2b2539b

Browse files
authored
ECHOES-1041 Increase SidebarNavigationGroup spacing with other elements (#603)
1 parent 804f9c2 commit 2b2539b

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

src/components/layout/sidebar-navigation/SidebarNavigationGroup.tsx

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ export const SidebarNavigationGroup = forwardRef<
4444
const id = `${useId()}-sidebar-nav-group`;
4545

4646
return (
47-
<UnstyledListItem>
47+
<SidebarNavigationGroupListItem>
4848
<SidebarNavigationGroupContainer
4949
aria-labelledby={id}
5050
className={className}
@@ -58,17 +58,27 @@ export const SidebarNavigationGroup = forwardRef<
5858
</SidebarNavigationGroupLabel>
5959
<SidebarNavigationGroupList>{children}</SidebarNavigationGroupList>
6060
</SidebarNavigationGroupContainer>
61-
</UnstyledListItem>
61+
</SidebarNavigationGroupListItem>
6262
);
6363
});
6464

6565
SidebarNavigationGroup.displayName = 'SidebarNavigationGroup';
6666

67+
const SidebarNavigationGroupListItem = styled(UnstyledListItem)`
68+
margin-top: ${cssVar('dimension-space-50')};
69+
margin-bottom: ${cssVar('dimension-space-100')};
70+
71+
// Reduce double margin when stacking groups, doesn't match figma for which it's too complex to implement
72+
& + & {
73+
margin-top: calc(-1 * ${cssVar('dimension-space-50')});
74+
}
75+
`;
76+
SidebarNavigationGroupListItem.displayName = 'SidebarNavigationGroupListItem';
77+
6778
const SidebarNavigationGroupContainer = styled.div`
6879
display: flex;
6980
flex-direction: column;
7081
gap: ${cssVar('dimension-space-50')};
71-
margin: ${cssVar('dimension-space-50')} 0;
7282
`;
7383
SidebarNavigationGroupContainer.displayName = 'SidebarNavigationGroupContainer';
7484

0 commit comments

Comments
 (0)