mirror/nanobot
1
0
Fork 0
mirror of https://github.com/HKUDS/nanobot.git synced 2026-09-02 17:22:06 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(webui): balance preset editor spacing

Browse Source
This commit is contained in:
Xubin Ren
2026-09-02 16:25:07 +08:00
parent 56699d4888
commit 6b385a1da6
2 changed files with 17 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
webui/src/components/settings/models/ModelsSettings.tsx
+6 -5
View File
@@ -372,7 +372,10 @@ export function ModelsSettings({
<div <div
id="model-preset-editor" id="model-preset-editor"
data-testid="model-preset-editor" data-testid="model-preset-editor"
className="mx-3 mb-3 divide-y divide-border/45 overflow-hidden rounded-floating border border-border/45 bg-background/80 shadow-sm motion-reduce:animate-none animate-in fade-in-0 slide-in-from-top-1 duration-200 sm:mx-5 lg:mx-auto lg:w-[calc(100%-2.5rem)] lg:max-w-6xl" className={cn(
"mx-3 mb-3 divide-y divide-border/45 overflow-hidden rounded-floating border border-border/45 bg-background/80 shadow-sm motion-reduce:animate-none animate-in fade-in-0 slide-in-from-top-1 duration-200 sm:mx-5 lg:mx-auto lg:w-[calc(100%-2.5rem)] lg:max-w-6xl",
creating && "mt-3",
)}
> >
{creating ? ( {creating ? (
<div className="flex min-h-[52px] items-center px-4 py-3 sm:px-5"> <div className="flex min-h-[52px] items-center px-4 py-3 sm:px-5">
@@ -846,8 +849,8 @@ export function ModelsSettings({
); );
})} })}
</div> </div>
<div className="flex min-h-[58px] flex-col gap-3 px-4 py-3 sm:flex-row sm:items-center sm:justify-between sm:px-5">
{!creating ? ( {!creating ? (
<div className="flex min-h-[58px] flex-col gap-3 px-4 py-3 sm:flex-row sm:items-center sm:justify-between sm:px-5">
<Button <Button
size="sm" size="sm"
variant="ghost" variant="ghost"
@@ -862,9 +865,6 @@ export function ModelsSettings({
<Plus className="mr-1.5 h-3.5 w-3.5" aria-hidden /> <Plus className="mr-1.5 h-3.5 w-3.5" aria-hidden />
{tx("settings.models.newPreset", "New model preset")} {tx("settings.models.newPreset", "New model preset")}
</Button> </Button>
) : (
<span />
)}
{orderSaving ? ( {orderSaving ? (
<SettingsStatusMessage> <SettingsStatusMessage>
<span className="inline-flex items-center gap-1.5"> <span className="inline-flex items-center gap-1.5">
@@ -874,6 +874,7 @@ export function ModelsSettings({
</SettingsStatusMessage> </SettingsStatusMessage>
) : null} ) : null}
</div> </div>
) : null}
{creating && editorOpen ? renderPresetEditor() : null} {creating && editorOpen ? renderPresetEditor() : null}
</> </>
)} )}
webui/src/tests/settings-models.test.tsx
+1
View File
@@ -1443,6 +1443,7 @@ describe("Settings models", () => {
fireEvent.click(await screen.findByRole("button", { name: "New model preset" })); fireEvent.click(await screen.findByRole("button", { name: "New model preset" }));
expect(screen.queryByRole("dialog", { name: "New model preset" })).not.toBeInTheDocument(); expect(screen.queryByRole("dialog", { name: "New model preset" })).not.toBeInTheDocument();
expect(screen.getByTestId("model-preset-editor")).toHaveClass("mt-3", "mb-3");
expect(screen.getByRole("textbox", { name: "Preset name" })).toHaveValue(""); expect(screen.getByRole("textbox", { name: "Preset name" })).toHaveValue("");
expect(screen.queryByText("Temperature")).not.toBeInTheDocument(); expect(screen.queryByText("Temperature")).not.toBeInTheDocument();
fireEvent.click(screen.getByRole("button", { name: /Advanced options/ })); fireEvent.click(screen.getByRole("button", { name: /Advanced options/ }));