Skip to content

Commit 05c78af

Browse files
andrewseguinAndrew Seguin
and
Andrew Seguin
committed
fix(cdk/table): run differ for all columns (#30012)
* fix(cdk/table): run differ for all columns * fix(cdk/table): include comment --------- Co-authored-by: Andrew Seguin <andrewseguin@google.com> (cherry picked from commit f1c4173)
1 parent 3a4158e commit 05c78af

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/cdk/table/table.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1085,7 +1085,12 @@ export class CdkTable<T>
10851085
* re-render that section.
10861086
*/
10871087
private _renderUpdatedColumns(): boolean {
1088-
const columnsDiffReducer = (acc: boolean, def: BaseRowDef) => acc || !!def.getColumnsDiff();
1088+
const columnsDiffReducer = (acc: boolean, def: BaseRowDef) => {
1089+
// The differ should be run for every column, even if `acc` is already
1090+
// true (see #29922)
1091+
const diff = !!def.getColumnsDiff();
1092+
return acc || diff;
1093+
};
10891094

10901095
// Force re-render data rows if the list of column definitions have changed.
10911096
const dataColumnsChanged = this._rowDefs.reduce(columnsDiffReducer, false);

0 commit comments

Comments
 (0)