From 6a9529aab7c75f0c7020ef88e41334018bba2fa5 Mon Sep 17 00:00:00 2001
From: ochafik <ochafik@google.com>
Date: Tue, 31 Dec 2024 16:20:34 +0000
Subject: [PATCH] Multimaterial: trim strings, skip empties

---
 src/components/MultimaterialColorsDialog.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/components/MultimaterialColorsDialog.tsx b/src/components/MultimaterialColorsDialog.tsx
index c995aa4..edb1d78 100644
--- a/src/components/MultimaterialColorsDialog.tsx
+++ b/src/components/MultimaterialColorsDialog.tsx
@@ -40,11 +40,11 @@ export default function MultimaterialColorsDialog() {
                     <Button 
                     label={state.view.extruderPickerVisibility == 'exporting' ? "Export" : "Save"}
                     icon="pi pi-check"
-                    disabled={!tempExtruderColors.every(c => chroma.valid(c))}
+                    disabled={!tempExtruderColors.every(c => chroma.valid(c) || c.trim() === '')}
                     autoFocus
                     onClick={e => {
                         model!.mutate(s => {
-                            s.params.extruderColors = tempExtruderColors;
+                            s.params.extruderColors = tempExtruderColors.filter(c => c.trim() !== '');
                             s.view.extruderPickerVisibility = undefined;
                         });
                         if (state.view.extruderPickerVisibility === 'exporting') {
@@ -84,9 +84,9 @@ export default function MultimaterialColorsDialog() {
                                         }
                                     }}
                                     onChange={(e) => {
-                                        let color = e.target.value;
+                                        let color = e.target.value.trim();
                                         try {
-                                            color = chroma(e.target.value).name();
+                                            color = chroma(color).name();
                                             console.log(`color: ${e.target.value} -> ${color}`);
                                         } catch (e) {
                                             // ignore