Skip to content

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Sep 4, 2020

Removes the APIs that were marked for removal in v11. There are also a few cases of the _animationMode parameter that I decided to keep optional since there's no good reason for it to be required.

BREAKING CHANGES:

  • The _element parameter has been removed from MatTabNav.updateActiveLink.
  • _platform parameter of the MatPaginatedTabHeader constructor is now required.
  • _document parameter of the MatTabBodyPortal constructor is now required.
  • platform parameter of the MatTabNav constructor is now required.
  • _closestTabGroup parameter of the MatTab constructor is now required.

@crisbeto crisbeto added P2 The issue is important to a large percentage of users, with a workaround target: major This PR is targeted for the next major release labels Sep 4, 2020
@crisbeto crisbeto added this to the 11.0.0 milestone Sep 4, 2020
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Sep 4, 2020
@crisbeto crisbeto force-pushed the tabs-v11-breaking-changes branch from 13fa73d to cb5bbb8 Compare September 4, 2020 14:50
Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

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

LGTM

@mmalerba mmalerba added the action: merge The PR is ready for merge by the caretaker label Sep 9, 2020
@crisbeto crisbeto force-pushed the tabs-v11-breaking-changes branch from cb5bbb8 to 4b05f98 Compare September 16, 2020 15:23
@wagnermaciel
Copy link
Contributor

@crisbeto Please resolve this merge conflict

@crisbeto crisbeto force-pushed the tabs-v11-breaking-changes branch from 4b05f98 to 3f954be Compare September 17, 2020 16:36
@crisbeto
Copy link
Member Author

Rebased.

Removes the APIs that were marked for removal in v11. There are also a few cases of the
`_animationMode` parameter that I decided to keep optional since there's no good reason for
it to be required.

BREAKING CHANGES:
* The `_element` parameter has been removed from `MatTabNav.updateActiveLink`.
* `_platform` parameter of the `MatPaginatedTabHeader` constructor is now required.
* `_document` parameter of the `MatTabBodyPortal` constructor is now required.
* `platform` parameter of the `MatTabNav` constructor is now required.
* `_closestTabGroup` parameter of the `MatTab` constructor is now required.
@crisbeto crisbeto force-pushed the tabs-v11-breaking-changes branch from 3f954be to 5a36b5c Compare September 17, 2020 18:54
@annieyw annieyw merged commit 9b47d3c into angular:master Sep 22, 2020
@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 Oct 23, 2020
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 cla: yes PR author has agreed to Google's Contributor License Agreement P2 The issue is important to a large percentage of users, with a workaround target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants