Skip to content

Commit 92448bf

Browse files
authored
Fix nav divider/kt (#3245)
* Fix divider spill * Remove hardcoded val. * Revert ai changes.
1 parent 1fcac13 commit 92448bf

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

src/lib/components/side-nav.svelte

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
<Navigation {isCloud} {bottom} aria-label={translate('common.primary')}>
1919
{#each linkList as item}
2020
{#if 'divider' in item && item.divider}
21-
<hr class="-mx-4 my-4 border-subtle" />
21+
<hr class="my-4 border-subtle group-data-[nav=closed]:hidden" />
2222
{:else if 'href' in item && !item.hidden}
2323
<NavigationItem
2424
link={item.href}

src/lib/holocene/navigation/navigation-container.svelte

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535

3636
<nav
3737
class={merge(
38-
'group grid min-h-screen w-16 grid-cols-[2rem] grid-rows-[fit-content(1.5rem)] gap-2 border-r border-subtle px-2 py-4 transition-width data-[nav=open]:w-auto data-[nav=open]:grid-cols-[100%]',
38+
'group grid min-h-screen grid-cols-[2rem] grid-rows-[fit-content(1.5rem)] gap-2 border-r border-subtle px-2 py-4 transition-width data-[nav=open]:w-auto data-[nav=open]:grid-cols-[100%]',
3939
'focus-visible:[&_[role=button]]:outline-none focus-visible:[&_[role=button]]:ring-2 focus-visible:[&_[role=button]]:ring-primary/70 focus-visible:[&_a]:outline-none focus-visible:[&_a]:ring-2 focus-visible:[&_a]:ring-primary/70',
4040
isCloud
4141
? 'w-16 bg-gradient-to-b from-indigo-600 to-indigo-950 text-off-white data-[nav=open]:w-[210px] focus-visible:[&_[role=button]]:outline-none focus-visible:[&_[role=button]]:ring-2 focus-visible:[&_[role=button]]:ring-success focus-visible:[&_a]:ring-success'

src/lib/holocene/navigation/navigation-item.svelte

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
data-track-intent="navigate"
4040
data-track-text={label}
4141
class={merge(
42-
'mb-1 flex items-center whitespace-nowrap px-2 py-1 text-sm ',
42+
'mb-1 flex items-center whitespace-nowrap px-2 py-1 text-sm',
4343
'hover:bg-black hover:bg-opacity-25 group-[.surface-black]:hover:bg-white group-[.surface-black]:hover:bg-opacity-25',
4444
active &&
4545
'bg-black bg-opacity-25 group-[.surface-black]:bg-white group-[.surface-black]:bg-opacity-25',

0 commit comments

Comments
 (0)