-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
feat(autocomplete): align with 2018 material design spec #12570
Conversation
@@ -579,6 +579,13 @@ export class MatAutocompleteTrigger implements ControlValueAccessor, OnDestroy { | |||
{originX: 'start', originY: 'top', overlayX: 'start', overlayY: 'bottom'} | |||
]); | |||
|
|||
this._positionStrategy.positionChanges.subscribe(({connectionPair}) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a comment here like...
// The overlay edge connected to the trigger has squared corners, while
// the opposite end has rounded corners. We apply a css class to swap the
// border-radius based on the overlay position.
Alternatively, since this seems to be a common pattern, what do you think of adding a panelClass
property to ConnectedPosition
(to be applied when that position is used)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've included the comment and I've left a note for myself to add a panelClass
to the ConnectedPosition
.
Aligns the autocomplete with the most-recent Material design spec. These changes are based on the menu designs since there isn't any concrete design for an autocomplete. Note that the changes only cover the adjustments to the autocomplete panel. The design changes for the options will come together with `mat-select` since the two components both use `mat-option`.
88566a8
to
a82ffe8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
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. |
Aligns the autocomplete with the most-recent Material design spec. These changes are based on the menu designs since there isn't any concrete design for an autocomplete. Note that the changes only cover the adjustments to the autocomplete panel. The design changes for the options will come together with
mat-select
since the two components both usemat-option
.