feat(autocomplete): add opened/closed events to panel #9904
Conversation
Adds the `opened` and `closed` event emitters to the autocomplete panel, allowing consumers to react to the panel being opened and closed. Fixes angular#9894.
googlebot
added
the
cla: yes
jelbourn
added
pr: lgtm
action: merge
| @@ -181,6 +181,7 @@ export class MatAutocompleteTrigger implements ControlValueAccessor, OnDestroy { | |||
|
|
|||
| if (this._panelOpen) { | |||
| this.autocomplete._isOpen = this._panelOpen = false; | |||
| this.autocomplete.closed.emit(); | |||
There was a problem hiding this comment.
Is it possible to emit the selected value if it's being closed as a result of a selection?
There was a problem hiding this comment.
You should be able to pick that one up from the optionSelected event.
There was a problem hiding this comment.
Yeah, I was kind of thinking of a way to implement (simplify my implementation of) the force selection. So if the closed emitted a null, I'd know that no option would have been selected. Without it, I think I'll have to combine closed with panelClosingActions. Something like (if panelClosingActions emits before closed):
this.trigger.autocomplete.closed
.pipe(
withLatestFrom(this.trigger.panelClosingActions)
)If the order of the events is inverted, I'll have to combine them in another way (maybe window operator?).
This dependency on the order of the emissions is not very comfortable.
|
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. |
Adds the
openedandclosedevent emitters to the autocomplete panel, allowing consumers to react to the panel being opened and closed.Fixes #9894.