Add 'hidden' optional property to DeclarativeTableColumn (#16386)

* add setHideColumns to DeclarativeTableComponent

* fixing tests using declarative table

* replace setHiddenColumns on DeclarativeTableComponent with 'hidden' optional property on DeclarativeTableColumn

* remove unnecessary changes
This commit is contained in:
Rachel Kim
2021-07-21 16:39:21 -07:00
committed by GitHub
parent c1f4c50177
commit d031211693
3 changed files with 9 additions and 2 deletions

View File

@@ -5,7 +5,8 @@
<ng-container *ngFor="let column of columns; let c = index;">
<th class="declarative-table-header" aria-sort="none" [style.width]="getColumnWidth(column)"
[ngClass]="{'declarative-table-cell-checkbox' : isCheckBox(c)}"
[ngStyle]="column.headerCssStyles" [attr.aria-label]="getHeaderAriaLabel(c)">
[ngStyle]="column.headerCssStyles" [attr.aria-label]="getHeaderAriaLabel(c)"
*ngIf="showColumn(column)">
{{column.displayName}}
<checkbox *ngIf="headerCheckboxVisible(c)" [checked]="isHeaderChecked(c)"
[aria-label]="getCheckAllColumnAriaLabel(c)" (onChange)="onHeaderCheckBoxChanged($event,c)"
@@ -22,7 +23,8 @@
<td class="declarative-table-cell" [style.width]="getColumnWidth(c)"
[attr.aria-label]="getAriaLabel(r, c)"
[ngClass]="{'declarative-table-cell-checkbox' : isCheckBox(c)}"
[ngStyle]="mergeCss(columns[c].rowCssStyles, cellData.style)" role="gridcell">
[ngStyle]="mergeCss(columns[c].rowCssStyles, cellData.style)" role="gridcell"
*ngIf="showColumn(columns[c])">
<checkbox *ngIf="isCheckBox(c)" label="" (onChange)="onCheckBoxChanged($event,r,c)"
[enabled]="isControlEnabled(r, c)" [checked]="isChecked(r,c)"
[ngStyle]="mergeCss(columns[c].rowCssStyles, cellData.style)">

View File

@@ -466,4 +466,8 @@ export default class DeclarativeTableComponent extends ContainerBase<any, azdata
this.setPropertyFromUI<number>((properties, value) => { properties.selectedRow = value; }, row);
}
}
public showColumn(column: azdata.DeclarativeTableColumn): boolean {
return column.hidden === undefined || !column.hidden;
}
}