Merge pull request #151 from satisfactorymodding/staging

fix: don't allow to select same tag multiple times
satisfactorymodding · Jan 7, 2024 · 1f0b08c · 1f0b08c
2 parents 7b75d64 + db2aaad
commit 1f0b08c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/lib/components/utils/TagList.svelte b/src/lib/components/utils/TagList.svelte
@@ -23,7 +23,8 @@
   $: allTags =
     $getAllTags.data?.getTags?.map((t) => ({
       label: `${t.name} - ${t.description}`,
-      value: t.id
+      value: t.id,
+      name: t.name
     })) || ([] satisfies AutocompleteOption[]);
 
   const popupSettings: PopupSettings = {
@@ -85,7 +86,7 @@
     <div class="card max-h-48 w-max max-w-full overflow-y-auto p-4" tabindex="-1" data-popup="popupAutocomplete">
       <Autocomplete
         bind:input={inputTag}
-        options={allTags.filter((t) => tagList.indexOf(t.label) < 0)}
+        options={allTags.filter((t) => tagList.indexOf(t.name) < 0)}
         on:selection={(t) => addTag(t.detail)} />
     </div>
   {/if}