refactor: simplify spacing between help menu items

This commit is contained in:
Carlos Valente
2026-05-13 20:05:06 +02:00
committed by Carlos Valente
parent 1dcb7fb08d
commit a0e7b9b665
3 changed files with 2 additions and 20 deletions
@@ -53,10 +53,6 @@ ul {
background-color: $red-400; background-color: $red-400;
} }
} }
&.split {
margin-top: 1rem;
}
} }
.secondary { .secondary {
@@ -42,16 +42,9 @@ interface PanelListItemProps {
location?: string; location?: string;
} }
function PanelListItem(props: PanelListItemProps) { function PanelListItem({ panel, isSelected, location }: PanelListItemProps) {
const { panel, isSelected, location } = props;
const { setLocation } = useAppSettingsNavigation(); const { setLocation } = useAppSettingsNavigation();
const classes = cx([style.primary, isSelected && style.active, panel.highlight && style.highlight]);
const classes = cx([
style.primary,
isSelected && style.active,
panel.split && style.split,
panel.highlight && style.highlight,
]);
return ( return (
<Fragment key={panel.id}> <Fragment key={panel.id}>
@@ -7,7 +7,6 @@ export type SettingsOption = {
id: string; id: string;
label: string; label: string;
secondary?: Readonly<SettingsOption[]>; secondary?: Readonly<SettingsOption[]>;
split?: boolean;
highlight?: string; highlight?: string;
}; };
@@ -26,7 +25,6 @@ const staticOptions = [
{ {
id: 'project', id: 'project',
label: 'Project', label: 'Project',
split: true,
secondary: [ secondary: [
{ id: 'project__create', label: 'Create...' }, { id: 'project__create', label: 'Create...' },
{ id: 'project__list', label: 'Manage projects' }, { id: 'project__list', label: 'Manage projects' },
@@ -46,7 +44,6 @@ const staticOptions = [
{ {
id: 'automation', id: 'automation',
label: 'Automation', label: 'Automation',
split: true,
secondary: [ secondary: [
{ id: 'automation__settings', label: 'Automation settings' }, { id: 'automation__settings', label: 'Automation settings' },
{ id: 'automation__automations', label: 'Manage automations' }, { id: 'automation__automations', label: 'Manage automations' },
@@ -56,7 +53,6 @@ const staticOptions = [
{ {
id: 'sharing', id: 'sharing',
label: 'Sharing and reporting', label: 'Sharing and reporting',
split: true,
secondary: [ secondary: [
{ id: 'sharing__presets', label: 'URL Presets' }, { id: 'sharing__presets', label: 'URL Presets' },
{ {
@@ -69,7 +65,6 @@ const staticOptions = [
{ {
id: 'network', id: 'network',
label: 'Network', label: 'Network',
split: true,
secondary: [ secondary: [
{ {
id: 'network__log', id: 'network__log',
@@ -84,12 +79,10 @@ const staticOptions = [
{ {
id: 'about', id: 'about',
label: 'About', label: 'About',
split: true,
}, },
{ {
id: 'shutdown', id: 'shutdown',
label: 'Shutdown', label: 'Shutdown',
split: true,
}, },
] as const; ] as const;