fix(sidebar): scope elevated mobile z-index to course sidebar (#720)

This commit is contained in:
Rotimi Best
2026-07-10 08:43:37 +01:00
committed by GitHub
2 changed files with 5 additions and 3 deletions
@@ -112,7 +112,7 @@
{#if !isOrgLoaded}
<SidebarSkeleton />
{:else}
<Sidebar.Root collapsible="icon" class="ui:z-150!">
<Sidebar.Root collapsible="icon" class="ui:z-150!" mobileOverlayClass="ui:z-150!">
<Sidebar.Header>
<CourseSidebarLogo />
</Sidebar.Header>
+4 -2
View File
@@ -11,12 +11,14 @@
variant = 'sidebar',
collapsible = 'offcanvas',
class: className,
mobileOverlayClass,
children,
...restProps
}: WithElementRef<HTMLAttributes<HTMLDivElement>> & {
side?: 'left' | 'right';
variant?: 'sidebar' | 'floating' | 'inset';
collapsible?: 'offcanvas' | 'icon' | 'none';
mobileOverlayClass?: string;
} = $props();
const sidebar = useSidebar();
@@ -39,9 +41,9 @@
data-sidebar="sidebar"
data-slot="sidebar"
data-mobile="true"
overlayClass="ui:z-150!"
overlayClass={mobileOverlayClass}
class={cn(
'ui:bg-sidebar ui:text-sidebar-foreground ui:w-(--sidebar-width) ui:z-150! ui:p-0 ui:[&>button]:hidden',
'ui:bg-sidebar ui:text-sidebar-foreground ui:w-(--sidebar-width) ui:p-0 ui:[&>button]:hidden',
className
)}
style="--sidebar-width: {SIDEBAR_WIDTH_MOBILE};"