From 5fb235f306fb10c0c4bedc2552a279c28dd4f1dd Mon Sep 17 00:00:00 2001 From: Simon Knott Date: Tue, 6 Aug 2024 12:56:17 +0200 Subject: [PATCH] remove unused `title` --- packages/web/src/components/splitView.tsx | 4 ++-- packages/web/src/uiUtils.ts | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/web/src/components/splitView.tsx b/packages/web/src/components/splitView.tsx index 8e2b242ff5..d8538d524e 100644 --- a/packages/web/src/components/splitView.tsx +++ b/packages/web/src/components/splitView.tsx @@ -43,8 +43,8 @@ export const SplitView: React.FC = ({ main, }) => { const defaultSize = Math.max(minSidebarSize, sidebarSize) * window.devicePixelRatio; - const [hSize, setHSize] = useSettingOrState(settingName + '.' + orientation + ':size', defaultSize, undefined, !!settingName); - const [vSize, setVSize] = useSettingOrState(settingName + '.' + orientation + ':size', defaultSize, undefined, !!settingName); + const [hSize, setHSize] = useSettingOrState(settingName + '.' + orientation + ':size', defaultSize, !!settingName); + const [vSize, setVSize] = useSettingOrState(settingName + '.' + orientation + ':size', defaultSize, !!settingName); const [resizing, setResizing] = React.useState<{ offset: number, size: number } | null>(null); const [measure, ref] = useMeasure(); diff --git a/packages/web/src/uiUtils.ts b/packages/web/src/uiUtils.ts index 509c9fc695..d53a0f55c5 100644 --- a/packages/web/src/uiUtils.ts +++ b/packages/web/src/uiUtils.ts @@ -157,10 +157,10 @@ export function useSetting(name: string, defaultValue: S, title?: string): [S return [value, setValueWrapper, setting]; } -export function useSettingOrState(name: string | undefined, defaultValue: S, title?: string, persist?: boolean): [S, (v: S) => void, Setting] { - const setting = useSetting(name ?? 'unused', defaultValue, title); +export function useSettingOrState(name: string | undefined, defaultValue: S, persist?: boolean): [S, (v: S) => void] { + const [settingValue, settingSet] = useSetting(name ?? 'unused', defaultValue); const state = React.useState(defaultValue); - return persist ? setting : [...state, [...state, title ?? '']]; + return persist ? [settingValue, settingSet] : state; } export class Settings {