chore(chip): remove redundant property

Tap30 · Jan 6, 2025 · e0a127a · e0a127a
1 parent 02c4169
commit e0a127a
Showing 1 changed file with 4 additions and 13 deletions.
diff --git a/packages/web-components/src/chip-group/chip-group.ts b/packages/web-components/src/chip-group/chip-group.ts
@@ -35,23 +35,14 @@ export class ChipGroup extends LitElement {
   @property({ type: Boolean, attribute: "full-width" })
   public fullWidth = false;
 
-  /**
-   * The selected chips elements.
-   */
-  public get selectedChips(): Chip[] {
-    if (isSSR()) return [];
-
-    return this._chips.filter(chip => chip.selected);
-  }
-
-  private get _chips() {
+  private get _selectedChips() {
     const chipsSlot = getRenderRootSlot(this.renderRoot, Slots.DEFAULT);
 
     if (!chipsSlot) return [];
 
     const chips = chipsSlot
       .assignedNodes()
-      .filter(node => node instanceof Chip);
+      .filter(node => node instanceof Chip && node.selected) as Chip[];
 
     return chips;
   }
@@ -97,7 +88,7 @@ export class ChipGroup extends LitElement {
     const chip = event.target as Chip;
     const value = chip.value;
 
-    const selectedValues = this.selectedChips
+    const selectedValues = this._selectedChips
       .map(chip => chip.value)
       .filter(v => v !== value);
 
@@ -112,7 +103,7 @@ export class ChipGroup extends LitElement {
     const chip = event.target as Chip;
     const value = chip.value;
 
-    const selectedValues = this.selectedChips
+    const selectedValues = this._selectedChips
       .map(chip => chip.value)
       .concat(value);