feat(components/switcher): move logic from ngonchanges to function #1476

zyfra · Jul 8, 2024 · d994b9d · d994b9d
1 parent 88739e2
commit d994b9d
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/libs/components/src/lib/components/switcher/switcher.component.ts b/libs/components/src/lib/components/switcher/switcher.component.ts
@@ -70,10 +70,7 @@ export class PrizmSwitcherComponent extends PrizmAbstractTestId implements Contr
  }
  ngOnChanges(changes: SimpleChanges): void {
  if (prizmHasChanges(changes, 'selectedSwitcherIdx') && prizmHasChanges(changes, 'switchers')) {
- const idx =
- this.selectedSwitcherIdx <= this.switchers.length ? this.selectedSwitcherIdx : INITIAL_SWITHCER_INDEX;
- const item = this.switchers[idx];
- if (item) this.selectSwitcher(item, idx);
+ this.setActiveSwitcherOnChange();
  }
  }
  public selectSwitcher(item: PrizmSwitcherItem, idx: number): void {
@@ -97,4 +94,11 @@ export class PrizmSwitcherComponent extends PrizmAbstractTestId implements Contr
  public setDisabledState(isDisabled: boolean): void {
  this.cdRef.markForCheck();
  }
+
+ private setActiveSwitcherOnChange(): void {
+ const idx =
+ this.selectedSwitcherIdx <= this.switchers.length ? this.selectedSwitcherIdx : INITIAL_SWITHCER_INDEX;
+ const item = this.switchers[idx];
+ if (item) this.selectSwitcher(item, idx);
+ }
 }