From 739ed86847b51a938e0c43f234f58ae13c293f2e Mon Sep 17 00:00:00 2001 From: "chembo.huang" Date: Wed, 18 Mar 2026 16:10:17 +0800 Subject: [PATCH] fix(ui): make ConfigProps fields optional to avoid upstream test breakage --- ui/src/ui/views/config.browser.test.ts | 2 ++ ui/src/ui/views/config.ts | 18 ++++++++++-------- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/ui/src/ui/views/config.browser.test.ts b/ui/src/ui/views/config.browser.test.ts index 6473c09404d..ebde01da266 100644 --- a/ui/src/ui/views/config.browser.test.ts +++ b/ui/src/ui/views/config.browser.test.ts @@ -46,6 +46,8 @@ describe("config view", () => { setBorderRadius: vi.fn(), gatewayUrl: "", assistantName: "OpenClaw", + borderRadius: 8, + setBorderRadius: vi.fn(), }); function findActionButtons(container: HTMLElement): { diff --git a/ui/src/ui/views/config.ts b/ui/src/ui/views/config.ts index 7c1121e6bb8..f2e32beddec 100644 --- a/ui/src/ui/views/config.ts +++ b/ui/src/ui/views/config.ts @@ -49,8 +49,8 @@ export type ConfigProps = { themeMode: ThemeMode; setTheme: (theme: ThemeName, context?: ThemeTransitionContext) => void; setThemeMode: (mode: ThemeMode, context?: ThemeTransitionContext) => void; - borderRadius: number; - setBorderRadius: (value: number) => void; + borderRadius?: number; + setBorderRadius?: (value: number) => void; gatewayUrl: string; assistantName: string; configPath?: string | null; @@ -517,6 +517,8 @@ const THEME_OPTIONS: ThemeOption[] = [ ]; function renderAppearanceSection(props: ConfigProps) { + const borderRadius = props.borderRadius ?? 8; + const setBorderRadius = props.setBorderRadius ?? (() => {}); return html`
@@ -559,7 +561,7 @@ function renderAppearanceSection(props: ConfigProps) { Square - ${props.borderRadius}% + ${borderRadius}% Round @@ -571,24 +573,24 @@ function renderAppearanceSection(props: ConfigProps) { min="0" max="100" step="1" - .value=${String(props.borderRadius)} + .value=${String(borderRadius)} @input=${(e: Event) => { const v = Number((e.target as HTMLInputElement).value); - props.setBorderRadius(v); + setBorderRadius(v); }} />