Skip to content

Commit 6c0e5a6

Browse files
committed
refactor(grid): reduce unwarranted deprecation warnings
1 parent bd8400c commit 6c0e5a6

File tree

5 files changed

+35
-8
lines changed

5 files changed

+35
-8
lines changed

projects/igniteui-angular/src/lib/grids/column-actions/column-actions.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ <h4 [attr.id]='titleID' class="igx-column-actions__header-title" *ngIf="title">{
1414
<div class="igx-column-actions__columns" tabindex="0"
1515
[style.max-height]="columnsAreaMaxHeight">
1616
<igx-checkbox
17-
*ngFor="let column of columns
17+
*ngFor="let column of grid?.columns
1818
| columnActionEnabled:actionsDirective.actionEnabledColumnsFilter:pipeTrigger
1919
| filterActionColumns:filterCriteria:pipeTrigger
2020
| sortActionColumns:columnDisplayOrder:pipeTrigger;"

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,7 @@ export class IgxColumnActionsComponent implements DoCheck {
366366
*/
367367
public ngDoCheck() {
368368
if (this._differ) {
369-
const changes = this._differ.diff(this.columns);
369+
const changes = this._differ.diff(this.grid?.columns);
370370
if (changes) {
371371
this._pipeTrigger++;
372372
}

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

Lines changed: 30 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -900,7 +900,18 @@ export abstract class IgxGridBaseDirective extends DisplayDensityBase implements
900900
*/
901901
@DeprecateProperty('`columnHidingTitle` is deprecated')
902902
@Input()
903-
public columnHidingTitle: string;
903+
public get columnHidingTitle(): string {
904+
return this._columnHidingTitle;
905+
}
906+
public set columnHidingTitle(v: string) {
907+
this._columnHidingTitle = v;
908+
}
909+
private _columnHidingTitle: string;
910+
911+
/** @hidden @internal */
912+
public get columnHidingTitleInternal(): string {
913+
return this._columnHidingTitle;
914+
}
904915

905916
/**
906917
* Gets/Sets the initial pinning configuration.
@@ -956,7 +967,18 @@ export abstract class IgxGridBaseDirective extends DisplayDensityBase implements
956967
*/
957968
@DeprecateProperty('`columnPinningTitle` is deprecated')
958969
@Input()
959-
public columnPinningTitle: string;
970+
public get columnPinningTitle(): string {
971+
return this._columnPinningTitle;
972+
}
973+
public set columnPinningTitle(v: string) {
974+
this._columnPinningTitle = v;
975+
}
976+
private _columnPinningTitle: string;
977+
978+
/** @hidden @internal */
979+
public get columnPinningTitleInternal(): string {
980+
return this._columnPinningTitle;
981+
}
960982

961983
/**
962984
* Gets/Sets if the filtering is enabled.
@@ -2198,7 +2220,7 @@ export abstract class IgxGridBaseDirective extends DisplayDensityBase implements
21982220
* <igx-grid [columnHiding]="true" [showToolbar]="true" [hiddenColumnsText]="'Hidden Columns'"></igx-grid>
21992221
* ```
22002222
*/
2201-
@DeprecateProperty('`hiddenColumnsText` is deprecated')
2223+
// @DeprecateProperty('`hiddenColumnsText` is deprecated')
22022224
@Input()
22032225
get hiddenColumnsText() {
22042226
return this._hiddenColumnsText;
@@ -2232,6 +2254,11 @@ export abstract class IgxGridBaseDirective extends DisplayDensityBase implements
22322254
this.notifyChanges();
22332255
}
22342256

2257+
/** @hidden @internal */
2258+
get pinnedColumnsTextInternal() {
2259+
return this._pinnedColumnsText;
2260+
}
2261+
22352262
/**
22362263
* Get transactions service for the grid.
22372264
*/

projects/igniteui-angular/src/lib/grids/toolbar/grid-toolbar-hiding.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
</div>
99
</button>
1010
<igx-column-actions #actions igxColumnHiding igxToggle (onOpened)="focusSearch(ref.element)" #ref="toggle"
11-
[grid]="grid" [title]="grid.columnHidingTitle || title || grid?.resourceStrings.igx_grid_toolbar_hiding_title"
11+
[grid]="grid" [title]="grid.columnHidingTitleInternal || title || grid?.resourceStrings.igx_grid_toolbar_hiding_title"
1212
[filterColumnsPrompt]="prompt || grid?.resourceStrings.igx_grid_toolbar_actions_filter_prompt">
1313
</igx-column-actions>
1414
</ng-container>

projects/igniteui-angular/src/lib/grids/toolbar/grid-toolbar-pinning.component.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,11 @@
44
<div class="igx-grid-toolbar__button-space">
55
<igx-icon fontSet="imx-icons" [name]="!grid.pinnedColumnsCount ? 'unpin-left' : 'pin-left'"></igx-icon>
66
<span>{{ grid.pinnedColumnsCount }}</span>
7-
<span>{{ grid.pinnedColumnsText }}</span>
7+
<span>{{ grid.pinnedColumnsTextInternal }}</span>
88
</div>
99
</button>
1010
<igx-column-actions #actions igxColumnPinning igxToggle (onOpened)="focusSearch(ref.element)" #ref="toggle"
11-
[grid]="grid" [title]="grid.columnPinningTitle || title || grid?.resourceStrings.igx_grid_toolbar_pinning_title"
11+
[grid]="grid" [title]="grid.columnPinningTitleInternal || title || grid?.resourceStrings.igx_grid_toolbar_pinning_title"
1212
[filterColumnsPrompt]="prompt || grid?.resourceStrings.igx_grid_toolbar_actions_filter_prompt">
1313
</igx-column-actions>
1414
</ng-container>

0 commit comments

Comments
 (0)