Chip workflow issue on remove

[MarkVasile]

When assembling chips using the chip component, not the chips aggregate, I ran into trouble trying to sync up the model upon removal of a chip.

<Chip v-for="tag in selectedTags" :label="tag" removable
          @remove="removeTag(tag)"
/>

With the above, I tried to update the selectedTags upon removal, but even though the selectedTags contains
the right elements (verified with console.log), I see 2 chips removed instead of 1.

function removeTag(tag: string) {
  const index = selectedTags.value.indexOf(tag)
  selectedTags.value.splice(index, 1)
  retrieveArticles()
}

I've also tried leaving the selectedTags alone, hoping the library will update the model, but it doesn't, i.e. the selectedTags stays intact after a chip removal.

Is there a different (correct) way to use the chip component I'm not aware of?

[tugcekucukoglu]

Could you share a stackblitz demo so we can easily replicate the problem?

[MarkVasile]

Ok, sorry I should've included the demo in my ticket. Unfortunately, the stackblitz won't let me share my code unless I pay, so I'm pasting the code here instead (please forgive me, i'm already paying for a few dozen dev services):

<template>
  <div id="app">
    <hr />
    <p>Selected tags</p>
    <Chip
      v-for="tag in selectedTags"
      :label="tag"
      removable
      @remove="removeTag(tag)"
    />

    <hr />

    <p>All tags</p>
    <Chip v-for="tag in tags" :label="tag" @click="selectTag(tag)" />
  </div>
</template>

<script>
import { ref } from 'vue'

const tags = ref(['beer', 'coffee', 'water'])
const selectedTags = ref([])

function selectTag(tag) {
  const seltags = selectedTags.value;
  if (!seltags.includes(tag)) seltags.push(tag)
}

function removeTag(tag) {
  const index = selectedTags.value.indexOf(tag)
  // ISSUE: without manually managing the tag list, the model does not get updated,
  // However, if we try to manually remove the tag, we get two tags removed instead of one.
  // Uncomment the following line to try manual removal:
  // selectedTags.value.splice(index, 1)
  console.log(`Tag removed ${tag}. Model updated: `, selectedTags.value)
}

export default {
  name: 'App',
  data() {
    return {
      tags,
      selectedTags,
    };
  },
  methods: {
    selectTag,
    removeTag,
  },
};
</script>

<style>
#app {
  font-family: Avenir, Helvetica, Arial, sans-serif;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
  text-align: center;
  color: white;
  margin-top: 60px;
}

.p-chip {
  cursor: pointer;
}
</style>

[jeremy-liss]

@MarkVasile did you solve this? I'm having the same issue.

[steve-t7]

This problem is still active @MarkVasile did you solved this, I ended up creating my own component

[j0rgedev]

Hello! I tried to replicate the issue, but didn't find any error. Could you please provide a minimal reproduction?
This is the tested code:

<template>
  <div class="card flex flex-wrap gap-12">
    <div class="flex flex-col">
      <p>Selected tags</p>
      <div class="flex flex-col">
        <Chip
          v-for="tag in selectedTags"
          :label="tag"
          :key="tag"
          removable
          @remove="removeTag(tag)"
        />
      </div>
    </div>

    <div class="flex flex-col">
      <p>All tags</p>
      <div class="flex flex-col">
        <Chip
          v-for="tag in tags"
          :key="tag"
          :label="tag"
          @click="selectTag(tag)"
        />
      </div>
    </div>
  </div>
</template>

<script setup>
import { ref } from 'vue';

const tags = ref(['beer', 'coffee', 'water']);
const selectedTags = ref([]);

function selectTag(tag) {
  const index = tags.value.indexOf(tag);

  if (index !== -1) {
    tags.value.splice(index, 1);

    selectedTags.value.push(tag);
  }
}

function removeTag(tag) {
  const index = selectedTags.value.indexOf(tag);

  if (index !== -1) {
    selectedTags.value.splice(index, 1);
  }
  console.log(`Tag removed ${tag}. Model updated: `, selectedTags.value);
}
</script>

<style>
.p-chip {
  cursor: pointer;
}
</style>