Navigation Panel UI Sizing Changes (#5964)
## Fixes #5902 : - [x] Navigation items' height should be risen to 28px. > For clarity: - [x] Also increased the height of NavigationDrawerSectionTitle to 28px to match navigation item. - [x] The gap between sections should be reduced to 12px > Was already completed it seems. - [x] The workspace switcher should be aligned with the navigation items --------- Co-authored-by: Lucas Bordeau <bordeau.lucas@gmail.com>
This commit is contained in:
@ -33,10 +33,12 @@ const StyledContainer = styled.div`
|
|||||||
border: 1px solid transparent;
|
border: 1px solid transparent;
|
||||||
display: flex;
|
display: flex;
|
||||||
justify-content: space-between;
|
justify-content: space-between;
|
||||||
height: ${({ theme }) => theme.spacing(7)};
|
height: ${({ theme }) => theme.spacing(5)};
|
||||||
padding: 0 ${({ theme }) => theme.spacing(2)};
|
padding: calc(${({ theme }) => theme.spacing(1)} - 1px);
|
||||||
width: 100%;
|
width: 100%;
|
||||||
|
|
||||||
|
gap: ${({ theme }) => theme.spacing(1)};
|
||||||
|
|
||||||
&:hover {
|
&:hover {
|
||||||
background-color: ${({ theme }) => theme.background.transparent.lighter};
|
background-color: ${({ theme }) => theme.background.transparent.lighter};
|
||||||
border: 1px solid ${({ theme }) => theme.border.color.medium};
|
border: 1px solid ${({ theme }) => theme.border.color.medium};
|
||||||
@ -46,7 +48,6 @@ const StyledContainer = styled.div`
|
|||||||
const StyledLabel = styled.div`
|
const StyledLabel = styled.div`
|
||||||
align-items: center;
|
align-items: center;
|
||||||
display: flex;
|
display: flex;
|
||||||
gap: ${({ theme }) => theme.spacing(2)};
|
|
||||||
`;
|
`;
|
||||||
|
|
||||||
const StyledIconChevronDown = styled(IconChevronDown)<{ disabled?: boolean }>`
|
const StyledIconChevronDown = styled(IconChevronDown)<{ disabled?: boolean }>`
|
||||||
|
|||||||
@ -53,7 +53,7 @@ const StyledContainer = styled.div<{ isSubMenu?: boolean }>`
|
|||||||
const StyledItemsContainer = styled.div`
|
const StyledItemsContainer = styled.div`
|
||||||
display: flex;
|
display: flex;
|
||||||
flex-direction: column;
|
flex-direction: column;
|
||||||
gap: ${({ theme }) => theme.spacing(8)};
|
gap: ${({ theme }) => theme.spacing(3)};
|
||||||
margin-bottom: auto;
|
margin-bottom: auto;
|
||||||
overflow-y: auto;
|
overflow-y: auto;
|
||||||
`;
|
`;
|
||||||
|
|||||||
@ -14,7 +14,7 @@ const StyledCollapseButton = styled.div`
|
|||||||
color: ${({ theme }) => theme.font.color.light};
|
color: ${({ theme }) => theme.font.color.light};
|
||||||
cursor: pointer;
|
cursor: pointer;
|
||||||
display: flex;
|
display: flex;
|
||||||
height: ${({ theme }) => theme.spacing(6)};
|
height: ${({ theme }) => theme.spacing(5)};
|
||||||
justify-content: center;
|
justify-content: center;
|
||||||
user-select: none;
|
user-select: none;
|
||||||
width: ${({ theme }) => theme.spacing(6)};
|
width: ${({ theme }) => theme.spacing(6)};
|
||||||
|
|||||||
@ -12,9 +12,7 @@ import { NavigationDrawerCollapseButton } from './NavigationDrawerCollapseButton
|
|||||||
const StyledContainer = styled.div`
|
const StyledContainer = styled.div`
|
||||||
align-items: center;
|
align-items: center;
|
||||||
display: flex;
|
display: flex;
|
||||||
gap: ${({ theme }) => theme.spacing(2)};
|
height: ${({ theme }) => theme.spacing(7)};
|
||||||
height: ${({ theme }) => theme.spacing(6)};
|
|
||||||
padding: ${({ theme }) => theme.spacing(1)};
|
|
||||||
user-select: none;
|
user-select: none;
|
||||||
`;
|
`;
|
||||||
|
|
||||||
|
|||||||
@ -39,6 +39,7 @@ const StyledItem = styled('div', {
|
|||||||
align-items: center;
|
align-items: center;
|
||||||
background: ${(props) =>
|
background: ${(props) =>
|
||||||
props.active ? props.theme.background.transparent.light : 'inherit'};
|
props.active ? props.theme.background.transparent.light : 'inherit'};
|
||||||
|
height: ${({ theme }) => theme.spacing(5)};
|
||||||
border: none;
|
border: none;
|
||||||
border-radius: ${({ theme }) => theme.border.radius.sm};
|
border-radius: ${({ theme }) => theme.border.radius.sm};
|
||||||
text-decoration: none;
|
text-decoration: none;
|
||||||
|
|||||||
@ -16,7 +16,7 @@ const StyledTitle = styled.div<{ onClick?: () => void }>`
|
|||||||
display: flex;
|
display: flex;
|
||||||
font-size: ${({ theme }) => theme.font.size.xs};
|
font-size: ${({ theme }) => theme.font.size.xs};
|
||||||
font-weight: ${({ theme }) => theme.font.weight.semiBold};
|
font-weight: ${({ theme }) => theme.font.weight.semiBold};
|
||||||
height: ${({ theme }) => theme.spacing(4)};
|
height: ${({ theme }) => theme.spacing(5)};
|
||||||
padding: ${({ theme }) => theme.spacing(1)};
|
padding: ${({ theme }) => theme.spacing(1)};
|
||||||
|
|
||||||
${({ onClick, theme }) =>
|
${({ onClick, theme }) =>
|
||||||
|
|||||||
Reference in New Issue
Block a user