Follow-up to #20634. Removes three desktop design changes that bled in unintentionally: - `font-weight: regular` → restored to `medium` on nav item labels - `MenuItemIconBoxContainer` wrap around bare icons → removed - Section title `padding-right/top` tweak → restored to original values Mobile-specific fixes from #20634 (slide-over drawer width, min-width overflow fixes, breadcrumb cleanup, etc.) are preserved.
This commit is contained in:
+13
-16
@@ -24,7 +24,6 @@ import {
|
||||
TooltipDelay,
|
||||
TooltipPosition,
|
||||
} from 'twenty-ui/display';
|
||||
import { MenuItemIconBoxContainer } from 'twenty-ui/navigation';
|
||||
import {
|
||||
MOBILE_VIEWPORT,
|
||||
ThemeContext,
|
||||
@@ -162,7 +161,7 @@ const StyledLabelParent = styled.div`
|
||||
`;
|
||||
|
||||
const StyledItemLabel = styled.span`
|
||||
font-weight: ${themeCssVariables.font.weight.regular};
|
||||
font-weight: ${themeCssVariables.font.weight.medium};
|
||||
`;
|
||||
|
||||
const StyledItemSecondaryLabel = styled.span`
|
||||
@@ -350,20 +349,18 @@ export const NavigationDrawerItem = ({
|
||||
</StyledIcon>
|
||||
) : (
|
||||
<StyledIcon>
|
||||
<MenuItemIconBoxContainer>
|
||||
<Icon
|
||||
style={{
|
||||
minWidth: theme.icon.size.md,
|
||||
}}
|
||||
size={theme.icon.size.md}
|
||||
stroke={theme.icon.stroke.md}
|
||||
color={
|
||||
showBreadcrumb && !isExpanded
|
||||
? theme.font.color.light
|
||||
: 'currentColor'
|
||||
}
|
||||
/>
|
||||
</MenuItemIconBoxContainer>
|
||||
<Icon
|
||||
style={{
|
||||
minWidth: theme.icon.size.md,
|
||||
}}
|
||||
size={theme.icon.size.md}
|
||||
stroke={theme.icon.stroke.md}
|
||||
color={
|
||||
showBreadcrumb && !isExpanded
|
||||
? theme.font.color.light
|
||||
: 'currentColor'
|
||||
}
|
||||
/>
|
||||
</StyledIcon>
|
||||
))}
|
||||
|
||||
|
||||
+2
-2
@@ -17,8 +17,8 @@ const StyledTitle = styled.div`
|
||||
justify-content: space-between;
|
||||
padding-bottom: ${themeCssVariables.spacing[1]};
|
||||
padding-left: ${themeCssVariables.spacing[1]};
|
||||
padding-right: ${themeCssVariables.spacing[1]};
|
||||
padding-top: ${themeCssVariables.spacing[2]};
|
||||
padding-right: ${themeCssVariables.spacing['0.5']};
|
||||
padding-top: ${themeCssVariables.spacing[1]};
|
||||
|
||||
&:hover {
|
||||
background-color: ${themeCssVariables.background.transparent.light};
|
||||
|
||||
Reference in New Issue
Block a user