Skip to content
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

fix(menu): not closed correctly if opened by different trigger while visible #15373

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Mar 3, 2019

Fixes the MatMenu overlay not being detached if the menu is opened by a different trigger while the panel is still open.

Fixes #15354.

@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Mar 3, 2019
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Mar 3, 2019
@lokeshdaiya
Copy link

When this PR can be merged ?

@mmalerba mmalerba added aaa and removed aaa labels Apr 25, 2019
@crisbeto crisbeto force-pushed the 15354/menu-multiple-trigger branch from 369ca01 to 4bc8f63 Compare May 2, 2019 06:26
@crisbeto crisbeto force-pushed the 15354/menu-multiple-trigger branch 2 times, most recently from 0c51b2c to d3e0f36 Compare May 24, 2019 05:36
@andrewseguin andrewseguin added the P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent label May 30, 2019
@crisbeto crisbeto force-pushed the 15354/menu-multiple-trigger branch from d3e0f36 to 949bd88 Compare July 18, 2019 21:03
@crisbeto crisbeto force-pushed the 15354/menu-multiple-trigger branch from f581ea3 to 06943e1 Compare April 17, 2020 18:50
…visible

Fixes the `MatMenu` overlay not being detached if the menu is opened by a different trigger while the panel is still open.

Fixes angular#15354.
@crisbeto crisbeto force-pushed the 15354/menu-multiple-trigger branch from 06943e1 to a431705 Compare July 25, 2020 15:45
@RadhikaGunisetty
Copy link

@crisbeto It looks like this PR is outdated, please rebase the PR.

@andrewseguin andrewseguin removed the cla: yes PR author has agreed to Google's Contributor License Agreement label Dec 29, 2021
@andrewseguin andrewseguin removed the P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent label Mar 28, 2022
@josephperrott josephperrott added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed needs rebase labels Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mat menu not working correctly when two triggers are used for same mat-menu
10 participants