Skip to content

Conversation

andrewseguin
Copy link
Contributor

Cloned from original author's change but is failing CLA #29922

This ensures the differ is run without short-circuiting and skipping columns, which is apparently important for change detection reasons

Fixes #29922

@andrewseguin andrewseguin requested a review from a team as a code owner November 13, 2024 20:31
@andrewseguin andrewseguin requested review from amysorto, crisbeto and wagnermaciel and removed request for a team November 13, 2024 20:31
@andrewseguin andrewseguin added the target: patch This PR is targeted for the next patch release label Nov 13, 2024
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Nov 14, 2024
@andrewseguin andrewseguin merged commit f1c4173 into angular:main Nov 14, 2024
21 of 23 checks passed
andrewseguin added a commit that referenced this pull request Nov 14, 2024
* fix(cdk/table): run differ for all columns

* fix(cdk/table): include comment

---------

Co-authored-by: Andrew Seguin <[email protected]>
(cherry picked from commit f1c4173)
andrewseguin added a commit that referenced this pull request Nov 14, 2024
* fix(cdk/table): run differ for all columns

* fix(cdk/table): include comment

---------

Co-authored-by: Andrew Seguin <[email protected]>
(cherry picked from commit f1c4173)
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CDK table inappropriately rerenders table header cells when there are multiple headers

2 participants