New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bug(Table): Angular 9: caching column definitions breaks nesting tables #18768
bug(Table): Angular 9: caching column definitions breaks nesting tables #18768
Comments
I'm facing this exact same issue with Angular 9 and Angular Material 9.1.0. Please help! |
@FredyValstrauss This component should work around the issue for the time being, though I trust the Angular team will be able to come up with a better solution. |
Hi @dtarjeft, your work aroud worked wonderful in my case. I only modified the template to use templateURL and use a separate html file, but it works just fine. As you said, this is a work around and Angular team should release a better and much simpler solution. Thanks again for the work around! |
Previously we used to support nesting tables, but in v9 we had to make some changes in order to handle all cases in Ivy. As a result, nesting was broken due to parent tables picking up the cell definitions of their children. These changes add some logic to account for tables being nested. Fixes angular#18768.
Previously we used to support nesting tables, but in v9 we had to make some changes in order to handle all cases in Ivy. As a result, nesting was broken due to parent tables picking up the cell definitions of their children. These changes add some logic to account for tables being nested. Fixes angular#18768.
Previously we used to support nesting tables, but in v9 we had to make some changes in order to handle all cases in Ivy. As a result, nesting was broken due to parent tables picking up the cell definitions of their children. These changes add some logic to account for tables being nested. Fixes angular#18768.
Previously we used to support nesting tables, but in v9 we had to make some changes in order to handle all cases in Ivy. As a result, nesting was broken due to parent tables picking up the cell definitions of their children. These changes add some logic to account for tables being nested. Fixes #18768.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Reproduction
Steps to reproduce:
Expected Behavior
Nested material tables in each mat-row with those columns defined, opening and closing as intended. No errors.
Actual Behavior
Expanding a row works the first time, then fails, leaving the table in a bad state throwing errors to the effect of
Duplicate column definition name provided
- the expansion/collapse functionality completely stops working at this point.One can work around this by creating a dummy component who's sole purpose is to pass the data to another instance of
mat-table
- I've shown this in a fork of the Angular 9 StackBlitz. Note: I didn't preserve the filters from the original StackBlitz, as that was not part of the functionality in question.Environment
The text was updated successfully, but these errors were encountered: