From 6e97ee5f675cf4f6fd7fd28f068b2bd6886eb8aa Mon Sep 17 00:00:00 2001
From: Diego Imbert <diego@windmill.dev>
Date: Sat, 27 Sep 2025 17:53:32 +0200
Subject: [PATCH] nits

---
 frontend/src/lib/components/InputTransformForm.svelte         | 4 +++-
 frontend/src/lib/components/S3FilePicker.svelte               | 2 +-
 frontend/src/lib/components/select/MultiSelect.svelte         | 2 +-
 frontend/src/lib/components/select/Select.svelte              | 2 +-
 frontend/src/lib/components/vscode.ts                         | 1 +
 .../lib/components/workspaceSettings/DucklakeSettings.svelte  | 2 +-
 6 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/frontend/src/lib/components/InputTransformForm.svelte b/frontend/src/lib/components/InputTransformForm.svelte
index bdec513aa8..c526781c0a 100644
--- a/frontend/src/lib/components/InputTransformForm.svelte
+++ b/frontend/src/lib/components/InputTransformForm.svelte
@@ -810,7 +810,9 @@
 
 						<div class="mb-2"></div>
 					{:else}
-						Not recognized input type {argName} ({arg.expr}, {propertyType})
+						<span class="text-xs text-red-500">
+							Not recognized input type {argName} ({arg.expr}, {propertyType})
+						</span>
 						<div class="flex mt-2">
 							<Button
 								variant="border"
diff --git a/frontend/src/lib/components/S3FilePicker.svelte b/frontend/src/lib/components/S3FilePicker.svelte
index db2b4a7b8e..c98725c287 100644
--- a/frontend/src/lib/components/S3FilePicker.svelte
+++ b/frontend/src/lib/components/S3FilePicker.svelte
@@ -819,7 +819,7 @@
 			<div class="flex gap-1">
 				{#if secondaryStorageNames.value?.length}
 					<Select
-						inputClass="h-10 min-w-44 !placeholder-tertiary"
+						inputClass="h-10 min-w-44 !placeholder-secondary"
 						items={[
 							{ value: undefined, label: 'Default storage' },
 							...secondaryStorageNames.value.map((value) => ({ value }))
diff --git a/frontend/src/lib/components/select/MultiSelect.svelte b/frontend/src/lib/components/select/MultiSelect.svelte
index f028f8482c..eff05d25d7 100644
--- a/frontend/src/lib/components/select/MultiSelect.svelte
+++ b/frontend/src/lib/components/select/MultiSelect.svelte
@@ -100,7 +100,7 @@
 <div
 	bind:this={wrapperEl}
 	class={twMerge(
-		'relative min-h-10 flex items-center w-full bg-surface-secondary text-tertiary rounded-md',
+		'relative min-h-10 flex items-center w-full bg-surface-secondary !text-secondary rounded-md',
 		inputBorderClass({ forceFocus: open && !disabled }),
 		disabled ? 'pointer-events-none' : '',
 		open && !disabled ? 'open' : '',
diff --git a/frontend/src/lib/components/select/Select.svelte b/frontend/src/lib/components/select/Select.svelte
index 55aab55da7..b2075c38aa 100644
--- a/frontend/src/lib/components/select/Select.svelte
+++ b/frontend/src/lib/components/select/Select.svelte
@@ -145,7 +145,7 @@
 			inputBorderClass({ error, forceFocus: open }),
 			'w-full',
 			open ? '' : 'cursor-pointer',
-			!loading && value ? 'placeholder-tertiary' : 'placeholder-hint',
+			!loading && value && !disabled ? '!placeholder-secondary' : 'placeholder-hint',
 			(clearable || RightIcon) && !disabled && value ? 'pr-8' : '',
 			inputClass ?? ''
 		)}
diff --git a/frontend/src/lib/components/vscode.ts b/frontend/src/lib/components/vscode.ts
index 3bcd116f1e..978b963ee9 100644
--- a/frontend/src/lib/components/vscode.ts
+++ b/frontend/src/lib/components/vscode.ts
@@ -245,6 +245,7 @@ export async function initializeVscode(caller?: string, htmlContainer?: HTMLElem
 				rules: [],
 				colors: {
 					'editor.background': '#FFFFFF00',
+					'editor.foreground': '#2d3748',
 					'editorLineNumber.foreground': '#C2C9D1',
 					'editorLineNumber.activeForeground': '#989DA5',
 					'editorGutter.background': '#FFFFFF00'
diff --git a/frontend/src/lib/components/workspaceSettings/DucklakeSettings.svelte b/frontend/src/lib/components/workspaceSettings/DucklakeSettings.svelte
index 193fcb68f3..1c6f8a93ea 100644
--- a/frontend/src/lib/components/workspaceSettings/DucklakeSettings.svelte
+++ b/frontend/src/lib/components/workspaceSettings/DucklakeSettings.svelte
@@ -290,7 +290,7 @@
 								}
 							}
 							class="w-48"
-							inputClass="!placeholder-tertiary"
+							inputClass="!placeholder-secondary"
 						/>
 						<TextInput
 							inputProps={{ placeholder: 'Data path (defaults to /)' }}