feat(autocomplete): make renderer constructor parameter required (#2621)

BREAKING CHANGE: `renderer` constructor parameter is now required.
akveo · Dec 16, 2020 · 2cd2c78 · 2cd2c78
1 parent 77a3519
commit 2cd2c78
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/src/framework/theme/components/autocomplete/autocomplete.directive.ts b/src/framework/theme/components/autocomplete/autocomplete.directive.ts
@@ -191,8 +191,7 @@ export class NbAutocompleteDirective<T> implements OnDestroy, AfterViewInit, Con
  protected triggerStrategyBuilder: NbTriggerStrategyBuilderService,
  protected positionBuilder: NbPositionBuilderService,
  protected activeDescendantKeyManagerFactory: NbActiveDescendantKeyManagerFactoryService<NbOptionComponent<T>>,
- // @breaking-change @7.0.0 Make renderer required.
- @Optional() protected renderer?: Renderer2,
+ protected renderer: Renderer2,
  ) {}
 
  ngAfterViewInit() {
@@ -265,12 +264,7 @@ export class NbAutocompleteDirective<T> implements OnDestroy, AfterViewInit, Con
  }
 
  setDisabledState(disabled: boolean): void {
- // @breaking-change @7.0.0 Keep only `this.renderer.setProperty` without `if/else`.
- if (this.renderer) {
- this.renderer.setProperty(this.hostRef.nativeElement, 'disabled', disabled);
- } else if (this.hostRef.nativeElement) {
- this.hostRef.nativeElement.disabled = disabled;
- }
+ this.renderer.setProperty(this.hostRef.nativeElement, 'disabled', disabled);
  }
 
  protected subscribeOnOptionClick() {