fix(vis): duplicate option values in scatterplot sidebar select (#261)

Co-authored-by: Michael Pühringer <[email protected]>
Co-authored-by: Holger Stitz <[email protected]>

src/vis/sidebar/MultiSelect.tsx

@@ -26,9 +26,9 @@ export function MultiSelect({
     callback(currentSelected.filter((s) => s.id !== id));
   };
 
-  const handleValueSelect = (name: string) => {
-    const itemToAdd = filteredColumns.find((c) => c.info.name === name);
-    currentSelected.find((c) => c.name === name) ? handleValueRemove(itemToAdd.info.id) : callback([...currentSelected, itemToAdd.info]);
+  const handleValueSelect = (id: string) => {
+    const itemToAdd = filteredColumns.find((c) => c.info.id === id);
+    currentSelected.find((c) => c.id === id) ? handleValueRemove(itemToAdd.info.id) : callback([...currentSelected, itemToAdd.info]);
   };
 
   const handleValueRemoveAll = () => {
@@ -39,7 +39,7 @@ export function MultiSelect({
     .filter((c) => !currentSelected.map((s) => s.id).includes(c.info.id))
     .map((item) => {
       return (
-        <Combobox.Option value={item.info.name} key={item.info.id}>
+        <Combobox.Option value={item.info.id} key={item.info.id}>
           <Tooltip
             withinPortal
             withArrow