Merge pull request #1128 from danielaskdd/main

Fix settings popover toggle logic
This commit is contained in:
Daniel.y
2025-03-20 04:49:56 +08:00
committed by GitHub
3 changed files with 38 additions and 40 deletions

View File

@@ -8,7 +8,7 @@
<link rel="icon" type="image/svg+xml" href="logo.png" /> <link rel="icon" type="image/svg+xml" href="logo.png" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> <meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Lightrag</title> <title>Lightrag</title>
<script type="module" crossorigin src="/webui/assets/index-BxHTAgSP.js"></script> <script type="module" crossorigin src="/webui/assets/index-CchWw5nW.js"></script>
<link rel="stylesheet" crossorigin href="/webui/assets/index-BMVv3U43.css"> <link rel="stylesheet" crossorigin href="/webui/assets/index-BMVv3U43.css">
</head> </head>
<body> <body>

View File

@@ -199,17 +199,15 @@ export default function Settings() {
const { t } = useTranslation(); const { t } = useTranslation();
const saveSettings = () => setOpened(false); const saveSettings = () => setOpened(false);
const toggleSettings = () => setOpened(!opened);
return ( return (
<> <>
<Popover open={opened}> <Popover open={opened} onOpenChange={setOpened}>
<PopoverTrigger asChild> <PopoverTrigger asChild>
<Button <Button
variant={controlButtonVariant} variant={controlButtonVariant}
tooltip={t('graphPanel.sideBar.settings.settings')} tooltip={t('graphPanel.sideBar.settings.settings')}
size="icon" size="icon"
onClick={toggleSettings}
> >
<SettingsIcon /> <SettingsIcon />
</Button> </Button>