Merge pull request #14966 from IgniteUI/bpachilova/14943-combo-id-setter-refactor-17.2.x

kacheshmarova · web-flow · commit 515bfdaaa5b9 · 2024-11-05T09:51:10.000+02:00
refactor(combo): only handle selection set in id setter - 17.2.x
diff --git a/projects/igniteui-angular/src/lib/combo/combo.common.ts b/projects/igniteui-angular/src/lib/combo/combo.common.ts
@@ -176,17 +176,11 @@ export abstract class IgxComboBaseDirective extends DisplayDensityBase implement
             return;
         }
         const selection = this.selectionService.get(this._id);
+        this.selectionService.clear(this._id);
         this._id = value;
         if (selection) {
             this.selectionService.set(this._id, selection);
         }
-        if (this.dropdown?.open) {
-            this.dropdown.close();
-        }
-        if (this.inputGroup?.isFocused) {
-            this.inputGroup.element.nativeElement.blur();
-            this.inputGroup.isFocused = false;
-        }
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -176,17 +176,11 @@ export abstract class IgxComboBaseDirective extends DisplayDensityBase implement`
`176`	`176`	`return;`
`177`	`177`	`}`
`178`	`178`	`const selection = this.selectionService.get(this._id);`
	`179`	`+ this.selectionService.clear(this._id);`
`179`	`180`	`this._id = value;`
`180`	`181`	`if (selection) {`
`181`	`182`	`this.selectionService.set(this._id, selection);`
`182`	`183`	`}`
`183`		`- if (this.dropdown?.open) {`
`184`		`- this.dropdown.close();`
`185`		`- }`
`186`		`- if (this.inputGroup?.isFocused) {`
`187`		`- this.inputGroup.element.nativeElement.blur();`
`188`		`- this.inputGroup.isFocused = false;`
`189`		`- }`
`190`	`184`	`}`
`191`	`185`
`192`	`186`	`/**`