refactor(ui): move About button back to user menu dropdown

This commit is contained in:
viettranx committed 2026-03-24 00:09:42 +07:00
1 parent fe26f2d3e0
commit 0bce64032a
1 file changed
+11 -12
+11 -12
View File
@@ -25,8 +25,6 @@ export function Topbar() {
const toggleSidebar = useUiStore((s) => s.toggleSidebar);
const setMobileSidebarOpen = useUiStore((s) => s.setMobileSidebarOpen);
const isMobile = useIsMobile();
const [showAbout, setShowAbout] = useState(false);
const isDark = theme === "dark" || (theme === "system" && window.matchMedia("(prefers-color-scheme: dark)").matches);
const handleSidebarToggle = isMobile
@@ -52,15 +50,6 @@ export function Topbar() {
</div>
<div className="flex items-center gap-2">
<button
onClick={() => setShowAbout(true)}
className="flex items-center gap-1 cursor-pointer rounded-md px-2 py-1.5 text-sm text-muted-foreground hover:bg-accent hover:text-accent-foreground"
title={t("about.menuItem")}
>
<Info className="h-4 w-4 shrink-0" />
<span className="hidden sm:inline">{t("about.menuItem")}</span>
</button>
<Select value={language} onValueChange={(v) => setLanguage(v as Language)}>
<SelectTrigger
title={t("language")}
@@ -102,7 +91,6 @@ export function Topbar() {
<UserMenu />
</div>
<AboutDialog open={showAbout} onOpenChange={setShowAbout} />
</header>
);
}
@@ -115,6 +103,7 @@ function UserMenu() {
const { currentTenant, currentTenantName, tenants, isCrossTenant, isMultiTenant, currentTenantId } = useTenants();
const [open, setOpen] = useState(false);
const [showLogoutConfirm, setShowLogoutConfirm] = useState(false);
const [showAbout, setShowAbout] = useState(false);
const navigate = useNavigate();
const tenantLabel = currentTenant?.name || currentTenantName || "";
@@ -206,6 +195,15 @@ function UserMenu() {
<span>{t("apiKeys")}</span>
</button>
{/* About */}
<button
onClick={() => { setOpen(false); setShowAbout(true); }}
className="flex w-full cursor-pointer items-center gap-2 rounded-md px-2 py-1.5 text-sm hover:bg-accent"
>
<Info className="h-3.5 w-3.5 shrink-0" />
<span>{t("about.menuItem")}</span>
</button>
<div className="my-1 border-t" />
{/* Logout */}
@@ -230,6 +228,7 @@ function UserMenu() {
onConfirm={() => { setShowLogoutConfirm(false); logout(); }}
/>
<AboutDialog open={showAbout} onOpenChange={setShowAbout} />
</>
);
}