From 3cf62c89db1b92ca550c0d2d41fc2b37c738d66d Mon Sep 17 00:00:00 2001
From: Jeff Gu <idevs@foxmail.com>
Date: Wed, 15 Apr 2026 10:39:05 +0800
Subject: [PATCH] =?UTF-8?q?feat(sidebar):=20=E6=B7=BB=E5=8A=A0=E4=B8=8B?=
 =?UTF-8?q?=E6=8B=89=E8=8F=9C=E5=8D=95=E4=BB=A5=E9=80=89=E6=8B=A9=E4=B8=BB?=
 =?UTF-8?q?=E9=A2=98=E5=92=8C=E8=AF=AD=E8=A8=80?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 frontend/package.json                        |   2 +
 frontend/src/components/layout/Sidebar.tsx   | 196 +++++++++++--------
 frontend/src/components/ui/dropdown-menu.tsx |  49 +++++
 frontend/src/components/ui/tooltip.tsx       |  34 ++++
 frontend/src/i18n/locales/en.json            |   2 +
 frontend/src/i18n/locales/zh-CN.json         |   2 +
 frontend/src/i18n/locales/zh-TW.json         |   2 +
 7 files changed, 203 insertions(+), 84 deletions(-)
 create mode 100644 frontend/src/components/ui/dropdown-menu.tsx
 create mode 100644 frontend/src/components/ui/tooltip.tsx

diff --git a/frontend/package.json b/frontend/package.json
index 2f78ce9..e673692 100644
--- a/frontend/package.json
+++ b/frontend/package.json
@@ -27,7 +27,9 @@
   },
   "dependencies": {
     "@radix-ui/react-dialog": "^1.1.15",
+    "@radix-ui/react-dropdown-menu": "^2.1.16",
     "@radix-ui/react-slot": "^1.2.4",
+    "@radix-ui/react-tooltip": "^1.2.8",
     "@tailwindcss/postcss": "^4.2.2",
     "@tanstack/react-query": "^5.97.0",
     "class-variance-authority": "^0.7.1",
diff --git a/frontend/src/components/layout/Sidebar.tsx b/frontend/src/components/layout/Sidebar.tsx
index 1ed7f2b..2d49572 100644
--- a/frontend/src/components/layout/Sidebar.tsx
+++ b/frontend/src/components/layout/Sidebar.tsx
@@ -11,6 +11,7 @@ import {
   Languages,
   Moon,
   Sun,
+  Check,
   LogOut,
   X,
   PanelLeftClose,
@@ -22,6 +23,13 @@ import { useAuthStore } from "@/store/auth";
 import { useState } from "react";
 import { useTranslation } from "react-i18next";
 import { changeLanguage, SUPPORTED_LANGUAGES, type SupportedLanguage } from "@/i18n";
+import {
+  DropdownMenu,
+  DropdownMenuContent,
+  DropdownMenuItem,
+  DropdownMenuTrigger,
+} from "@/components/ui/dropdown-menu";
+import { Tooltip, TooltipContent, TooltipProvider, TooltipTrigger } from "@/components/ui/tooltip";
 
 type NavGroup = "overview" | "management" | "tools" | "system";
 
@@ -77,11 +85,10 @@ export function Sidebar({
     return "light";
   });
 
-  const toggleTheme = () => {
-    const newTheme = theme === "light" ? "dark" : "light";
-    setTheme(newTheme);
-    localStorage.setItem("theme", newTheme);
-    if (newTheme === "dark") {
+  const setThemeMode = (nextTheme: "light" | "dark") => {
+    setTheme(nextTheme);
+    localStorage.setItem("theme", nextTheme);
+    if (nextTheme === "dark") {
       document.documentElement.classList.add("dark");
     } else {
       document.documentElement.classList.remove("dark");
@@ -91,16 +98,17 @@ export function Sidebar({
   const langLabels: Record<SupportedLanguage, string> = {
     "zh-CN": t("language.zhCN"),
     "zh-TW": t("language.zhTW"),
-    "en": t("language.en"),
+    en: t("language.en"),
   };
 
-  const cycleLang = () => {
-    const currentIndex = SUPPORTED_LANGUAGES.indexOf(i18n.language as SupportedLanguage);
-    const nextIndex = (currentIndex + 1) % SUPPORTED_LANGUAGES.length;
-    changeLanguage(SUPPORTED_LANGUAGES[nextIndex]);
-  };
-
-  const currentLangLabel = langLabels[i18n.language as SupportedLanguage] || langLabels["zh-CN"];
+  const currentLanguage = SUPPORTED_LANGUAGES.includes(i18n.language as SupportedLanguage)
+    ? (i18n.language as SupportedLanguage)
+    : "zh-CN";
+  const currentLangLabel = langLabels[currentLanguage];
+  const actionButtonClassName = cn(
+    "flex h-9 items-center justify-center rounded-md text-muted-foreground transition-colors duration-150 hover:bg-muted/70 hover:text-foreground focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring/50 data-[state=open]:bg-muted/80 data-[state=open]:text-foreground",
+    collapsed ? "w-10" : "w-full",
+  );
 
   const handleLogout = async () => {
     await logout();
@@ -185,7 +193,10 @@ export function Sidebar({
       {/* Navigation */}
       <nav className="flex-1 px-2.5 py-3 overflow-y-auto scrollbar-thin">
         {groupOrder.map((group) => (
-          <div key={group} className={cn("space-y-1", group !== groupOrder[0] && "mt-3 pt-3 border-t border-border/30")}>
+          <div
+            key={group}
+            className={cn("space-y-1", group !== groupOrder[0] && "mt-3 pt-3 border-t border-border/30")}
+          >
             {groupedNavItems[group].map((item) => (
               <NavLink key={item.href} item={item} />
             ))}
@@ -196,83 +207,100 @@ export function Sidebar({
       {/* Footer */}
       <div className={cn("p-2.5 border-t", collapsed && "p-2")}>
         {/* Action Buttons */}
-        <div className={cn("flex gap-1 mb-2", collapsed ? "flex-col items-center" : "")}>
-          <button
-            type="button"
-            title={collapsed ? (theme === "dark" ? t('sidebar.lightMode') : t('sidebar.darkMode')) : undefined}
+        <TooltipProvider delayDuration={120}>
+          <div
             className={cn(
-              "flex items-center justify-center rounded-md transition-colors duration-150",
-              collapsed
-                ? "w-10 h-9 hover:bg-muted text-muted-foreground hover:text-foreground"
-                : "flex-1 h-8 gap-1.5 px-2.5 hover:bg-muted/70 text-muted-foreground hover:text-foreground text-[11px] font-medium",
-            )}
-            onClick={toggleTheme}
-          >
-            {theme === "dark" ? <Sun className="h-3.5 w-3.5" /> : <Moon className="h-3.5 w-3.5" />}
-            {!collapsed && <span>{theme === "dark" ? t('sidebar.lightMode') : t('sidebar.darkMode')}</span>}
-          </button>
-
-          <button
-            type="button"
-            title={collapsed ? currentLangLabel : undefined}
-            className={cn(
-              "flex items-center justify-center rounded-md transition-colors duration-150",
-              collapsed
-                ? "w-10 h-9 hover:bg-muted text-muted-foreground hover:text-foreground"
-                : "flex-1 h-8 gap-1.5 px-2.5 hover:bg-muted/70 text-muted-foreground hover:text-foreground text-[11px] font-medium",
-            )}
-            onClick={cycleLang}
-          >
-            <Languages className="h-3.5 w-3.5" />
-            {!collapsed && <span>{currentLangLabel}</span>}
-          </button>
-
-          <a
-            href="https://github.com/Tokinx/one-api-workers"
-            target="_blank"
-            rel="noopener noreferrer"
-            title={collapsed ? "GitHub" : undefined}
-            className={cn(
-              "flex items-center justify-center rounded-md transition-colors duration-150",
-              collapsed
-                ? "w-10 h-9 hover:bg-muted text-muted-foreground hover:text-foreground"
-                : "flex-1 h-8 gap-1.5 px-2.5 hover:bg-muted/70 text-muted-foreground hover:text-foreground text-[11px] font-medium",
+              "mb-2 grid gap-1.5",
+              collapsed ? "grid-cols-1 justify-items-center" : showCollapseToggle ? "grid-cols-4" : "grid-cols-3",
             )}
           >
-            <GitBranch className="h-3.5 w-3.5" />
-            {!collapsed && <span>GitHub</span>}
-          </a>
+            <Tooltip>
+              <TooltipTrigger asChild>
+                <a
+                  href="https://github.com/Tokinx/one-api-workers"
+                  target="_blank"
+                  rel="noopener noreferrer"
+                  aria-label="GitHub"
+                  className={actionButtonClassName}
+                >
+                  <GitBranch className="h-3.5 w-3.5" />
+                </a>
+              </TooltipTrigger>
+              <TooltipContent side="top">GitHub</TooltipContent>
+            </Tooltip>
 
-          {/* Collapse Toggle */}
-          {showCollapseToggle && (
-            <div className={cn("hidden lg:flex", collapsed && "px-0 justify-center")}>
-              <button
-                type="button"
-                onClick={onToggleCollapse}
-                className={cn(
-                  "flex items-center justify-center rounded-md transition-colors duration-150",
-                  collapsed
-                    ? "w-10 h-9 hover:bg-muted text-muted-foreground hover:text-foreground"
-                    : "flex-1 h-8 gap-1.5 px-2.5 hover:bg-muted/70 text-muted-foreground hover:text-foreground text-[11px] font-medium",
-                )}
-              >
-                {collapsed ? (
-                  <PanelLeftOpen className="h-3.5 w-3.5" />
-                ) : (
-                  <>
-                    <PanelLeftClose className="h-3.5 w-3.5" />
-                    <span>{t('sidebar.collapse')}</span>
-                  </>
-                )}
-              </button>
-            </div>
-          )}
-        </div>
+            <DropdownMenu>
+              <DropdownMenuTrigger asChild>
+                <button
+                  type="button"
+                  aria-label={`${t("sidebar.theme")} · ${theme === "dark" ? t("sidebar.darkMode") : t("sidebar.lightMode")}`}
+                  className={actionButtonClassName}
+                >
+                  {theme === "dark" ? <Moon className="h-3.5 w-3.5" /> : <Sun className="h-3.5 w-3.5" />}
+                </button>
+              </DropdownMenuTrigger>
+              <DropdownMenuContent side="top" align="center" className="w-40">
+                <DropdownMenuItem className="justify-between" onSelect={() => setThemeMode("light")}>
+                  <span className="flex items-center gap-2">
+                    <Sun className="h-3.5 w-3.5" />
+                    {t("sidebar.lightMode")}
+                  </span>
+                  {theme === "light" && <Check className="h-3.5 w-3.5 text-primary" />}
+                </DropdownMenuItem>
+                <DropdownMenuItem className="justify-between" onSelect={() => setThemeMode("dark")}>
+                  <span className="flex items-center gap-2">
+                    <Moon className="h-3.5 w-3.5" />
+                    {t("sidebar.darkMode")}
+                  </span>
+                  {theme === "dark" && <Check className="h-3.5 w-3.5 text-primary" />}
+                </DropdownMenuItem>
+              </DropdownMenuContent>
+            </DropdownMenu>
+
+            <DropdownMenu>
+              <DropdownMenuTrigger asChild>
+                <button
+                  type="button"
+                  aria-label={`${t("sidebar.language")} · ${currentLangLabel}`}
+                  className={actionButtonClassName}
+                >
+                  <Languages className="h-3.5 w-3.5" />
+                </button>
+              </DropdownMenuTrigger>
+              <DropdownMenuContent side="top" align="center" className="w-40">
+                {SUPPORTED_LANGUAGES.map((lang) => (
+                  <DropdownMenuItem key={lang} className="justify-between" onSelect={() => changeLanguage(lang)}>
+                    <span>{langLabels[lang]}</span>
+                    {currentLanguage === lang && <Check className="h-3.5 w-3.5 text-primary" />}
+                  </DropdownMenuItem>
+                ))}
+              </DropdownMenuContent>
+            </DropdownMenu>
+
+            {showCollapseToggle && (
+              <Tooltip>
+                <TooltipTrigger asChild>
+                  <button
+                    type="button"
+                    onClick={onToggleCollapse}
+                    aria-label={collapsed ? t("sidebar.openSidebar") : t("sidebar.collapse")}
+                    className={actionButtonClassName}
+                  >
+                    {collapsed ? <PanelLeftOpen className="h-3.5 w-3.5" /> : <PanelLeftClose className="h-3.5 w-3.5" />}
+                  </button>
+                </TooltipTrigger>
+                <TooltipContent side="top">
+                  {collapsed ? t("sidebar.openSidebar") : t("sidebar.collapse")}
+                </TooltipContent>
+              </Tooltip>
+            )}
+          </div>
+        </TooltipProvider>
 
         {/* Logout Button */}
         <button
           type="button"
-          title={collapsed ? t('sidebar.logout') : undefined}
+          title={collapsed ? t("sidebar.logout") : undefined}
           className={cn(
             "flex items-center justify-center gap-2 w-full rounded-md text-[13px] font-medium transition-all duration-150",
             collapsed ? "h-9" : "h-9 px-3",
@@ -281,7 +309,7 @@ export function Sidebar({
           onClick={handleLogout}
         >
           <LogOut className="h-3.5 w-3.5" />
-          {!collapsed && <span>{t('sidebar.logout')}</span>}
+          {!collapsed && <span>{t("sidebar.logout")}</span>}
         </button>
       </div>
     </aside>
diff --git a/frontend/src/components/ui/dropdown-menu.tsx b/frontend/src/components/ui/dropdown-menu.tsx
new file mode 100644
index 0000000..8ce9f1b
--- /dev/null
+++ b/frontend/src/components/ui/dropdown-menu.tsx
@@ -0,0 +1,49 @@
+import * as React from "react"
+import * as DropdownMenuPrimitive from "@radix-ui/react-dropdown-menu"
+import { cn } from "@/lib/utils"
+
+const DropdownMenu = DropdownMenuPrimitive.Root
+
+const DropdownMenuTrigger = DropdownMenuPrimitive.Trigger
+
+const DropdownMenuPortal = DropdownMenuPrimitive.Portal
+
+const DropdownMenuContent = React.forwardRef<
+  React.ElementRef<typeof DropdownMenuPrimitive.Content>,
+  React.ComponentPropsWithoutRef<typeof DropdownMenuPrimitive.Content>
+>(({ className, sideOffset = 8, ...props }, ref) => (
+  <DropdownMenuPortal>
+    <DropdownMenuPrimitive.Content
+      ref={ref}
+      sideOffset={sideOffset}
+      className={cn(
+        "z-50 min-w-40 overflow-hidden rounded-md border border-border bg-popover p-1 text-popover-foreground shadow-xl data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 data-[side=bottom]:slide-in-from-top-1 data-[side=left]:slide-in-from-right-1 data-[side=right]:slide-in-from-left-1 data-[side=top]:slide-in-from-bottom-1",
+        className
+      )}
+      {...props}
+    />
+  </DropdownMenuPortal>
+))
+DropdownMenuContent.displayName = DropdownMenuPrimitive.Content.displayName
+
+const DropdownMenuItem = React.forwardRef<
+  React.ElementRef<typeof DropdownMenuPrimitive.Item>,
+  React.ComponentPropsWithoutRef<typeof DropdownMenuPrimitive.Item>
+>(({ className, ...props }, ref) => (
+  <DropdownMenuPrimitive.Item
+    ref={ref}
+    className={cn(
+      "relative flex cursor-default select-none items-center gap-2 rounded-sm px-2.5 py-2 text-sm outline-none transition-colors duration-150 focus:bg-muted/80 focus:text-foreground data-[disabled]:pointer-events-none data-[disabled]:opacity-50 [&_svg]:pointer-events-none [&_svg]:size-4 [&_svg]:shrink-0",
+      className
+    )}
+    {...props}
+  />
+))
+DropdownMenuItem.displayName = DropdownMenuPrimitive.Item.displayName
+
+export {
+  DropdownMenu,
+  DropdownMenuTrigger,
+  DropdownMenuContent,
+  DropdownMenuItem,
+}
diff --git a/frontend/src/components/ui/tooltip.tsx b/frontend/src/components/ui/tooltip.tsx
new file mode 100644
index 0000000..c9181ae
--- /dev/null
+++ b/frontend/src/components/ui/tooltip.tsx
@@ -0,0 +1,34 @@
+import * as React from "react"
+import * as TooltipPrimitive from "@radix-ui/react-tooltip"
+import { cn } from "@/lib/utils"
+
+const TooltipProvider = TooltipPrimitive.Provider
+
+const Tooltip = TooltipPrimitive.Root
+
+const TooltipTrigger = TooltipPrimitive.Trigger
+
+const TooltipContent = React.forwardRef<
+  React.ElementRef<typeof TooltipPrimitive.Content>,
+  React.ComponentPropsWithoutRef<typeof TooltipPrimitive.Content>
+>(({ className, sideOffset = 8, ...props }, ref) => (
+  <TooltipPrimitive.Portal>
+    <TooltipPrimitive.Content
+      ref={ref}
+      sideOffset={sideOffset}
+      className={cn(
+        "z-50 overflow-hidden rounded-md border border-border bg-popover px-2.5 py-1.5 text-xs font-medium text-popover-foreground shadow-lg data-[state=delayed-open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=delayed-open]:fade-in-0 data-[side=bottom]:slide-in-from-top-1 data-[side=left]:slide-in-from-right-1 data-[side=right]:slide-in-from-left-1 data-[side=top]:slide-in-from-bottom-1",
+        className
+      )}
+      {...props}
+    />
+  </TooltipPrimitive.Portal>
+))
+TooltipContent.displayName = TooltipPrimitive.Content.displayName
+
+export {
+  TooltipProvider,
+  Tooltip,
+  TooltipTrigger,
+  TooltipContent,
+}
diff --git a/frontend/src/i18n/locales/en.json b/frontend/src/i18n/locales/en.json
index d1a6575..c630112 100644
--- a/frontend/src/i18n/locales/en.json
+++ b/frontend/src/i18n/locales/en.json
@@ -53,6 +53,8 @@
     "usageLogs": "Usage Logs",
     "apiTest": "API Test",
     "settings": "Settings",
+    "theme": "Theme",
+    "language": "Language",
     "lightMode": "Light",
     "darkMode": "Dark",
     "collapse": "Collapse",
diff --git a/frontend/src/i18n/locales/zh-CN.json b/frontend/src/i18n/locales/zh-CN.json
index d6d1c7f..48ae2ff 100644
--- a/frontend/src/i18n/locales/zh-CN.json
+++ b/frontend/src/i18n/locales/zh-CN.json
@@ -53,6 +53,8 @@
     "usageLogs": "使用日志",
     "apiTest": "API 测试",
     "settings": "系统设置",
+    "theme": "主题",
+    "language": "语言",
     "lightMode": "浅色",
     "darkMode": "深色",
     "collapse": "收起",
diff --git a/frontend/src/i18n/locales/zh-TW.json b/frontend/src/i18n/locales/zh-TW.json
index 2b177b8..1c2db8d 100644
--- a/frontend/src/i18n/locales/zh-TW.json
+++ b/frontend/src/i18n/locales/zh-TW.json
@@ -53,6 +53,8 @@
     "usageLogs": "使用日誌",
     "apiTest": "API 測試",
     "settings": "系統設定",
+    "theme": "主題",
+    "language": "語言",
     "lightMode": "淺色",
     "darkMode": "深色",
     "collapse": "收起",