From aa4ce3478a432b90d5878e4c428c8f8e2410eda2 Mon Sep 17 00:00:00 2001
From: Andrew <andrewpareles@gmail.com>
Date: Wed, 16 Oct 2024 23:03:51 -0700
Subject: [PATCH] update styles in settings page

---
 .../void/src/sidebar/SidebarSettings.tsx      | 115 +++++++++++-------
 .../void/src/sidebar/contextForConfig.tsx     |   2 +-
 2 files changed, 72 insertions(+), 45 deletions(-)

diff --git a/extensions/void/src/sidebar/SidebarSettings.tsx b/extensions/void/src/sidebar/SidebarSettings.tsx
index a8dc040d..47d657f0 100644
--- a/extensions/void/src/sidebar/SidebarSettings.tsx
+++ b/extensions/void/src/sidebar/SidebarSettings.tsx
@@ -5,49 +5,55 @@ import { configFields, useVoidConfig, VoidConfigField } from "./contextForConfig
 const SettingOfFieldAndParam = ({ field, param }: { field: VoidConfigField, param: string }) => {
 	const { voidConfig, partialVoidConfig, voidConfigInfo, setConfigParam } = useVoidConfig()
 
+	const { defaultVal, description } = voidConfigInfo[field][param]
 	const [val, setVal] = useState<string | undefined>(partialVoidConfig[field]?.[param])
 
 	const { enumArr } = voidConfigInfo[field][param]
 
-	const updateState = (e: React.ChangeEvent<HTMLInputElement | HTMLSelectElement>) => { setVal(e.target.value); };
-	const commitConfigParam = () => { if (val) setConfigParam(field, param, val); };
+	const updateState = (newValue: string | undefined) => {
+		setVal(newValue)
+		if (newValue)
+			setConfigParam(field, param, newValue)
+	}
 
 
-	// string
-	if (enumArr === undefined) {
-		return (
-			<div>
-				<label>{param}</label>
-				<input
-					className='input'
-					type="text"
-					value={val}
-					onChange={updateState}
-					onBlur={commitConfigParam}
-				/>
-			</div>
-		)
-	}
-	// enum
-	else {
-		return (
-			<div>
-				<label>{param}</label>
-				<select
-				className='dropdown'
-					value={val}
-					onChange={updateState}
-					onBlur={commitConfigParam}
-				>
-					{enumArr.map((option) => (
-						<option key={option} value={option}>
-							{option}
-						</option>
-					))}
-				</select>
-			</div>
-		)
-	}
+	const resetButton = <button className='btn btn-sm' onClick={() => updateState(defaultVal)}>
+		<svg
+			className='size-5'
+			stroke="currentColor" fill="currentColor" strokeWidth="0" viewBox="0 0 16 16" height="200px" width="200px" xmlns="http://www.w3.org/2000/svg"><path fillRule="evenodd" clipRule="evenodd" d="M3.5 2v3.5L4 6h3.5V5H4.979l.941-.941a3.552 3.552 0 1 1 5.023 5.023L5.746 14.28l.72.72 5.198-5.198A4.57 4.57 0 0 0 5.2 3.339l-.7.7V2h-1z"></path>
+		</svg>
+	</button>
+
+	const inputElement = enumArr === undefined ?
+		// string
+		(<input
+			className='input p-1 w-full'
+			type="text"
+			value={val}
+			onChange={(e) => updateState(e.target.value)}
+		/>)
+		:
+		// enum
+		(<select
+			className='dropdown p-1 w-full'
+			value={val}
+			onChange={(e) => updateState(e.target.value)}
+		>
+			{enumArr.map((option) => (
+				<option key={option} value={option}>
+					{option}
+				</option>
+			))}
+		</select>)
+
+	return <div>
+		<label className='hidden'>{param}</label>
+		<span>{description}</span>
+		<div className='flex items-center'>
+			{inputElement}
+			{resetButton}
+		</div>
+	</div>
 }
 
 export const SidebarSettings = () => {
@@ -55,17 +61,38 @@ export const SidebarSettings = () => {
 	const { voidConfig, voidConfigInfo } = useVoidConfig()
 
 	const current_field = voidConfig.default['whichApi'] as VoidConfigField
-	const params = Object.keys(voidConfigInfo[current_field])
+
 
 	return (
-		<div>
-			{params.map((param) => (
+		<div className='space-y-4 py-2 overflow-y-auto'>
+
+			{/* choose the field */}
+			<div className='outline-vscode-input-bg'>
 				<SettingOfFieldAndParam
-					key={param}
-					field={current_field}
-					param={param}
+					field='default'
+					param='whichApi'
 				/>
-			))}
+			</div>
+
+			<hr />
+
+			{/* render all fields, but hide the ones not visible for fast tab switching */}
+			{configFields.map(field => {
+				return <div
+					key={field}
+					className={`flex flex-col gap-y-2 ${field !== current_field ? 'hidden' : ''}`}
+				>
+					{Object.keys(voidConfigInfo[field]).map((param) => (
+						<SettingOfFieldAndParam
+							key={param}
+							field={field}
+							param={param}
+						/>
+					))}
+				</div>
+			})}
+
+
 		</div>
 	)
 }
diff --git a/extensions/void/src/sidebar/contextForConfig.tsx b/extensions/void/src/sidebar/contextForConfig.tsx
index 14b1cce7..30c2219d 100644
--- a/extensions/void/src/sidebar/contextForConfig.tsx
+++ b/extensions/void/src/sidebar/contextForConfig.tsx
@@ -41,7 +41,7 @@ const voidConfigInfo: Record<
 > = {
 	default: {
 		whichApi: configEnum(
-			"Choose an API provider.",
+			"API Provider.",
 			'anthropic',
 			configFields,
 		),