Skip to content

fix(menu): closing parent overlay when pressing escape - #13442

Merged
josephperrott merged 1 commit into
angular:masterfrom
crisbeto:menu-overlay-keydown-events
Oct 19, 2018
Merged

fix(menu): closing parent overlay when pressing escape#13442
josephperrott merged 1 commit into
angular:masterfrom
crisbeto:menu-overlay-keydown-events

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Along the same lines as #13413. Since MatMenu doesn't consume the OverlayRef.keydownEvents, it means that they'll be passed along to the next overlay in the stack.

@crisbetocrisbeto added the target: patch This PR is targeted for the next patch release label Oct 5, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 5, 2018
Along the same lines as angular#13413. Since `MatMenu` doesn't consume the `OverlayRef.keydownEvents`, it means that they'll be passed along to the next overlay in the stack.
@crisbeto
crisbetoforce-pushed the menu-overlay-keydown-events branch from d013491 to 0da21c5CompareOctober 5, 2018 12:19

@jelbournjelbourn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jelbournjelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Oct 5, 2018
@josephperrott
josephperrott merged commit a7df1d0 into angular:masterOct 19, 2018
@angular-automatic-lock-bot

Copy link
Copy Markdown

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-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 9, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreementtarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@crisbeto@jelbourn@googlebot@josephperrott