feat: enhance workspace navigation menu with conditional rendering and mounted state

This commit is contained in:
Henry Li
2026-02-07 11:10:24 +08:00
parent 60be7ee20d
commit 91a05acdf8

View File

@@ -9,7 +9,7 @@ import {
Settings2Icon,
SettingsIcon,
} from "lucide-react";
import { useState } from "react";
import { useEffect, useState } from "react";
import {
DropdownMenu,
@@ -30,29 +30,14 @@ import { useI18n } from "@/core/i18n/hooks";
import { GithubIcon } from "./github-icon";
import { SettingsDialog } from "./settings";
export function WorkspaceNavMenu() {
const [settingsOpen, setSettingsOpen] = useState(false);
const [settingsDefaultSection, setSettingsDefaultSection] = useState<
"appearance" | "memory" | "tools" | "skills" | "notification" | "about"
>("appearance");
const { open: isSidebarOpen } = useSidebar();
const { t } = useI18n();
return (
<>
<SettingsDialog
open={settingsOpen}
onOpenChange={setSettingsOpen}
defaultSection={settingsDefaultSection}
/>
<SidebarMenu className="w-full">
<SidebarMenuItem>
<DropdownMenu>
<DropdownMenuTrigger asChild>
<SidebarMenuButton
size="lg"
className="data-[state=open]:bg-sidebar-accent data-[state=open]:text-sidebar-accent-foreground"
>
{isSidebarOpen ? (
function NavMenuButtonContent({
isSidebarOpen,
t,
}: {
isSidebarOpen: boolean;
t: ReturnType<typeof useI18n>["t"];
}) {
return isSidebarOpen ? (
<div className="text-muted-foreground flex w-full items-center gap-2 text-left text-sm">
<SettingsIcon className="size-4" />
<span>{t.workspace.settingsAndMore}</span>
@@ -62,7 +47,39 @@ export function WorkspaceNavMenu() {
<div className="flex size-full items-center justify-center">
<SettingsIcon className="text-muted-foreground size-4" />
</div>
)}
);
}
export function WorkspaceNavMenu() {
const [settingsOpen, setSettingsOpen] = useState(false);
const [settingsDefaultSection, setSettingsDefaultSection] = useState<
"appearance" | "memory" | "tools" | "skills" | "notification" | "about"
>("appearance");
const [mounted, setMounted] = useState(false);
const { open: isSidebarOpen } = useSidebar();
const { t } = useI18n();
useEffect(() => {
setMounted(true);
}, []);
return (
<>
<SettingsDialog
open={settingsOpen}
onOpenChange={setSettingsOpen}
defaultSection={settingsDefaultSection}
/>
<SidebarMenu className="w-full">
<SidebarMenuItem>
{mounted ? (
<DropdownMenu>
<DropdownMenuTrigger asChild>
<SidebarMenuButton
size="lg"
className="data-[state=open]:bg-sidebar-accent data-[state=open]:text-sidebar-accent-foreground"
>
<NavMenuButtonContent isSidebarOpen={isSidebarOpen} t={t} />
</SidebarMenuButton>
</DropdownMenuTrigger>
<DropdownMenuContent
@@ -131,6 +148,11 @@ export function WorkspaceNavMenu() {
</DropdownMenuItem>
</DropdownMenuContent>
</DropdownMenu>
) : (
<SidebarMenuButton size="lg" className="pointer-events-none">
<NavMenuButtonContent isSidebarOpen={isSidebarOpen} t={t} />
</SidebarMenuButton>
)}
</SidebarMenuItem>
</SidebarMenu>
</>