chore(*): address comments

Author: RivaIvanova

projects/igniteui-angular/src/lib/grids/columns/column.component.ts

@@ -2154,8 +2154,7 @@ export class IgxColumnComponent implements AfterContentInit, OnDestroy, ColumnTy
             if (size && !!size.width) {
                 result.push(size.width + 'px');
             } else {
-                const possibleWidth = this.getPossibeWidth();
-                const currentWidth = parseFloat(possibleWidth);
+                const currentWidth = parseFloat(this.grid.getPossibleColumnWidth());
                 result.push((this.getConstrainedSizePx(currentWidth)) + 'px');
             }
         }
@@ -2650,12 +2649,18 @@ export class IgxColumnComponent implements AfterContentInit, OnDestroy, ColumnTy
             this._calcWidth = this.grid.calcWidth ? this.getConstrainedSizePx(currentCalcWidth) : 0;
         } else if (!colWidth || isAutoWidth && !this.autoSize) {
             // no width
-            const possibleWidth = this.getPossibeWidth();
-            const currentCalcWidth = this.defaultWidth || possibleWidth;
+            const currentCalcWidth = this.defaultWidth || this.grid.getPossibleColumnWidth();
             this._calcWidth = this.getConstrainedSizePx(currentCalcWidth);
         } else {
-            const currentCalcWidth =  parseFloat(this.width);
-            this._calcWidth =this.getConstrainedSizePx(currentCalcWidth);
+            let possibleColumnWidth = '';
+            if (!this.widthSetByUser && this.minWidthSetByUser && this.userSetMinWidthPx < this.grid.minColumnWidth) {
+                possibleColumnWidth = this.defaultWidth = this.grid.getPossibleColumnWidth(null, this.userSetMinWidthPx);
+            } else {
+                possibleColumnWidth = this.width;
+            }
+
+            const currentCalcWidth = parseFloat(possibleColumnWidth);
+            this._calcWidth = this.getConstrainedSizePx(currentCalcWidth);
         }
         this.calcPixelWidth = parseFloat(this._calcWidth);
     }
@@ -2720,12 +2725,4 @@ export class IgxColumnComponent implements AfterContentInit, OnDestroy, ColumnTy
     public get minWidthSetByUser(): boolean {
         return this._minWidthSetByUser;
     }
-
-    private getPossibeWidth(): string {
-        const possibleWidth = this.grid.getPossibleColumnWidth();
-        if (!this.minWidthSetByUser && parseFloat(possibleWidth) < this.grid.minColumnWidth) {
-            return this.grid.minColumnWidth + 'px';
-        }
-        return possibleWidth;
-    }
 }

projects/igniteui-angular/src/lib/grids/common/grid.interface.ts

@@ -1156,7 +1156,7 @@ export interface GridType extends IGridDataBindable {
     refreshSearch(): void;
     getDefaultExpandState(record: any): boolean;
     trackColumnChanges(index: number, column: any): any;
-    getPossibleColumnWidth(): string;
+    getPossibleColumnWidth(baseWidth?: number, minColumnWidth?: number): string;
     resetHorizontalVirtualization(): void;
     hasVerticalScroll(): boolean;
     getVisibleContentHeight(): number;

projects/igniteui-angular/src/lib/grids/grid-base.directive.ts

@@ -3237,7 +3237,6 @@ export abstract class IgxGridBaseDirective implements GridType,
     private _shouldRecalcRowHeight = false;
 
     private _hostWidth;
-    private _possibleColumnWidth: string;
     private _advancedFilteringOverlayId: string;
     private _advancedFilteringPositionSettings: PositionSettings = {
         verticalDirection: VerticalAlignment.Middle,
@@ -5485,7 +5484,7 @@ export abstract class IgxGridBaseDirective implements GridType,
     /**
      * @hidden @internal
      */
-    public getPossibleColumnWidth(baseWidth: number = null) {
+    public getPossibleColumnWidth(baseWidth: number = null, minColumnWidth: number = null) {
         let computedWidth;
         if (baseWidth !== null) {
             computedWidth = baseWidth;
@@ -5534,9 +5533,11 @@ export abstract class IgxGridBaseDirective implements GridType,
         }
         computedWidth -= this.featureColumnsWidth();
 
+        const minColWidth = minColumnWidth || this.minColumnWidth;
+
         const columnWidth = !Number.isFinite(sumExistingWidths) ?
-            Math.max(computedWidth / columnsToSize, this.defaultHeaderGroupMinWidth) :
-            Math.max((computedWidth - sumExistingWidths) / columnsToSize, this.defaultHeaderGroupMinWidth);
+            Math.max(computedWidth / columnsToSize, minColWidth) :
+            Math.max((computedWidth - sumExistingWidths) / columnsToSize, minColWidth);
 
         return columnWidth + 'px';
     }
@@ -6620,10 +6621,9 @@ export abstract class IgxGridBaseDirective implements GridType,
      */
     protected _derivePossibleWidth() {
         if (!this.columnWidthSetByUser) {
-            this._possibleColumnWidth = this._columnWidth = this.width !== null ? this.getPossibleColumnWidth() : this.minColumnWidth + 'px';
+            this._columnWidth = this.width !== null ? this.getPossibleColumnWidth() : this.minColumnWidth + 'px';
         }
         this._columns.forEach((column: IgxColumnComponent) => {
-            this.setPossibleMinColumnWidth(column);
             if (this.hasColumnLayouts && parseFloat(this._columnWidth)) {
                 const columnWidthCombined = parseFloat(this._columnWidth) * (column.colEnd ? column.colEnd - column.colStart : 1);
                 column.defaultWidth = columnWidthCombined + 'px';
@@ -8010,16 +8010,4 @@ export abstract class IgxGridBaseDirective implements GridType,
         }
         this._rowCount += 1; // include header row
     }
-
-    private setPossibleMinColumnWidth(column: IgxColumnComponent): void {
-        if (this.columnWidthSetByUser || this.width == null) {
-            return;
-        }
-
-        if (column.minWidthSetByUser || column.hidden) {
-            this._columnWidth = this._possibleColumnWidth;
-        } else if (!column.minWidthSetByUser && parseFloat(this._possibleColumnWidth) < this.minColumnWidth) {
-            this._columnWidth = this.minColumnWidth + 'px';
-        }
-    }
 }