Skip to content

Conversation

crisbeto
Copy link
Member

Switches mat-select to use MDC by default.

BREAKING CHANGE:

  • DOM and CSS classes for mat-select have changes.
  • TypeScript API is largely the same but may have minor differences.

Switches `mat-select` to use MDC by default.

BREAKING CHANGE:

* DOM and CSS classes for `mat-select` have changes.
* TypeScript API is largely the same but may have minor differences.
@crisbeto crisbeto force-pushed the mdc-select-move-2 branch from e3474df to 918d952 Compare July 30, 2022 07:11
@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 Jul 30, 2022
@crisbeto crisbeto force-pushed the mdc-select-move-2 branch from c4b83a8 to 918d952 Compare July 30, 2022 10:22
@crisbeto crisbeto marked this pull request as ready for review July 30, 2022 11:18
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Aug 1, 2022
@crisbeto crisbeto self-assigned this Aug 1, 2022
@crisbeto crisbeto merged commit bf6f38a into angular:main Aug 1, 2022
*/

/** The max height of the select's overlay panel. */
export const SELECT_PANEL_MAX_HEIGHT = 256;
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we meant to prefix these exports with LEGACY_ as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

My understanding was that we're only supposed to do the Mat* symbols, but I could be wrong.

@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 Sep 1, 2022
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 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.

3 participants