fix(datagrid): skip listening when datagrid have static height

Author: Daniel Tsanev

projects/angular/clarity.api.md

@@ -1249,14 +1249,16 @@ export class ClrDatagridCell implements OnInit {
 export class ClrDatagridColumn<T = any> extends DatagridFilterRegistrar<T, ClrDatagridFilterInterface<T>> implements OnDestroy, OnInit, OnChanges {
     // Warning: (ae-forgotten-export) The symbol "Sort" needs to be exported by the entry point index.d.ts
     // Warning: (ae-forgotten-export) The symbol "FiltersProvider" needs to be exported by the entry point index.d.ts
-    constructor(_sort: Sort<T>, filters: FiltersProvider<T>, vcr: ViewContainerRef, detailService: DetailService, changeDetectorRef: ChangeDetectorRef, smartPositionService: ClrPopoverPositionService, smartEventsService: ClrPopoverEventsService);
+    constructor(_sort: Sort<T>, filters: FiltersProvider<T>, vcr: ViewContainerRef, detailService: DetailService, changeDetectorRef: ChangeDetectorRef, smartPositionService: ClrPopoverPositionService, smartEventsService: ClrPopoverEventsService, elementRef: ElementRef);
     // (undocumented)
     get ariaSort(): "none" | "ascending" | "descending";
     // (undocumented)
     get colType(): 'string' | 'number';
     set colType(value: 'string' | 'number');
     customFilter: boolean;
     // (undocumented)
+    get datagridElementRef(): any;
+    // (undocumented)
     get field(): string;
     set field(field: string);
     // (undocumented)

projects/angular/src/data/datagrid/datagrid-column.ts

@@ -9,6 +9,7 @@ import {
   ChangeDetectorRef,
   Component,
   ContentChild,
+  ElementRef,
   EventEmitter,
   Injector,
   Input,
@@ -20,7 +21,7 @@ import {
   ViewContainerRef,
 } from '@angular/core';
 import { Subscription } from 'rxjs';
-import { debounceTime, tap } from 'rxjs/operators';
+import { debounceTime, skipWhile, tap } from 'rxjs/operators';
 
 import { ClrPopoverEventsService, ClrPopoverPositionService } from '../../utils';
 import { HostWrapper } from '../../utils/host-wrapping/host-wrapper';
@@ -153,7 +154,8 @@ export class ClrDatagridColumn<T = any>
     private detailService: DetailService,
     private changeDetectorRef: ChangeDetectorRef,
     private smartPositionService: ClrPopoverPositionService,
-    private smartEventsService: ClrPopoverEventsService
+    private smartEventsService: ClrPopoverEventsService,
+    private elementRef: ElementRef
   ) {
     super(filters);
     this.subscriptions.push(this.listenForSortingChanges());
@@ -311,6 +313,10 @@ export class ClrDatagridColumn<T = any>
     return this.wrappedInjector.get(WrappedColumn, this.vcr).columnView;
   }
 
+  get datagridElementRef() {
+    return this.elementRef?.nativeElement?.closest('clr-datagrid');
+  }
+
   ngOnInit() {
     this.wrappedInjector = new HostWrapper(WrappedColumn, this.vcr);
   }
@@ -381,6 +387,7 @@ export class ClrDatagridColumn<T = any>
   private listenForFilterChanges() {
     return this.filter.changes
       .pipe(
+        skipWhile(() => !!this.datagridElementRef?.style.height === true),
         tap(() => {
           this.smartEventsService.removeScrollListener();
           this.smartPositionService.realign();