Skip to content

Conversation

crisbeto
Copy link
Member

Fixes that the flexbox-based table had a hardcoded height for the rows, instead of using the tokens.

Fixes #30906.

Fixes that the flexbox-based table had a hardcoded height for the rows, instead of using the tokens.

Fixes angular#30906.
@crisbeto crisbeto added the target: major This PR is targeted for the next major release label Apr 29, 2025
@crisbeto crisbeto requested a review from a team as a code owner April 29, 2025 13:06
@crisbeto crisbeto requested review from adolgachev and andrewseguin and removed request for a team April 29, 2025 13:06
@melroy89
Copy link

Thanks for the PR.

@crisbeto crisbeto added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker and removed target: major This PR is targeted for the next major release labels Apr 29, 2025
@crisbeto crisbeto removed the request for review from adolgachev April 29, 2025 18:00
@crisbeto crisbeto merged commit 50a9a9b into angular:main Apr 29, 2025
32 checks passed
@crisbeto
Copy link
Member Author

The changes were merged into the following branches: main, 19.2.x

crisbeto added a commit that referenced this pull request Apr 29, 2025
Fixes that the flexbox-based table had a hardcoded height for the rows, instead of using the tokens.

Fixes #30906.

(cherry picked from commit 50a9a9b)
@melroy89
Copy link

There is a regression in your CI/CD now?

 Error: Can't find stylesheet to import.
  ╷
2 │ @use './m2-table';
  │ ^^^^^^^^^^^^^^^^^

crisbeto added a commit to crisbeto/material2 that referenced this pull request Apr 29, 2025
angular#30978 was cherry-picked into the patch branch, but it's using APIs that aren't available in v19. These changes resolve the build failure.
@crisbeto crisbeto mentioned this pull request Apr 29, 2025
crisbeto added a commit that referenced this pull request Apr 29, 2025
#30978 was cherry-picked into the patch branch, but it's using APIs that aren't available in v19. These changes resolve the build failure.
wildcardalice pushed a commit to wildcardalice/components that referenced this pull request May 12, 2025
…r#30978)

Fixes that the flexbox-based table had a hardcoded height for the rows, instead of using the tokens.

Fixes angular#30906.
@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 May 30, 2025
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 area: material/table target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(mat-table): Unable to use mat-table-row-item-container-height

3 participants