Skip to content

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Apr 2, 2025

Fixes a long-standing issue where users weren't able to assign an ID to a tab.

Fixes #4136.

Fixes a long-standing issue where users weren't able to assign an ID to a tab.

Fixes angular#4136.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Apr 2, 2025
@crisbeto crisbeto requested a review from a team as a code owner April 2, 2025 07:49
@crisbeto crisbeto requested review from mmalerba and wagnermaciel and removed request for a team April 2, 2025 07:49
@crisbeto crisbeto requested a review from andrewseguin April 2, 2025 07:51
* Custom ID for the tab, overriding the auto-generated one by Material.
* Note that when using this input, it's your responsibility to ensure that the ID is unique.
*/
@Input() id: string | null = null;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this is a new input, and no one is currently depending on it, wouldn't it be better if it was a signal input?

@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Apr 2, 2025
@crisbeto crisbeto merged commit 387313f into angular:main Apr 2, 2025
23 of 25 checks passed
@crisbeto
Copy link
Member Author

crisbeto commented Apr 2, 2025

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

crisbeto added a commit that referenced this pull request Apr 2, 2025
Fixes a long-standing issue where users weren't able to assign an ID to a tab.

Fixes #4136.

(cherry picked from commit 387313f)
mistrykaran91 pushed a commit to mistrykaran91/components that referenced this pull request Apr 7, 2025
Fixes a long-standing issue where users weren't able to assign an ID to a tab.

Fixes angular#4136.
@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 3, 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/tabs target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Tabs] Allow users to assign IDs to tabs

3 participants