fix: require a filename for save as dialog

leaphy-robotics · May 9, 2024 · a178c73 · a178c73
1 parent eb8789f
commit a178c73
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/lib/components/core/header/Header.svelte b/src/lib/components/core/header/Header.svelte
@@ -99,6 +99,7 @@ async function saveProjectAs() {
 			name: "SAVEAS",
 			placeholder: "GIVE_FILENAME",
 			confirm: "SAVE",
+			requireValue: true,
 		},
 		allowInteraction: false,
 	});

diff --git a/src/lib/components/core/popups/popups/Prompt.svelte b/src/lib/components/core/popups/popups/Prompt.svelte
@@ -11,15 +11,23 @@ interface Props {
 	placeholder: string;
 	confirm: string;
 	value?: string;
+	requireValue: boolean;
 }
-let { name, placeholder, confirm, value = $bindable("") }: Props = $props();
+let {
+	name,
+	placeholder,
+	confirm,
+	value = $bindable(""),
+	requireValue = false,
+}: Props = $props();
 const popupState = getContext<Writable<PopupState>>("state");
 
 function cancel() {
 	popups.close($popupState.id, false);
 }
 
 function save() {
+	if (requireValue && !value) return;
 	popups.close($popupState.id, value);
 }