Skip to content

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

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

Closed
wants to merge 1 commit into 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 949bd88 to 078951d Compare October 1, 2019 19:31
@crisbeto crisbeto force-pushed the 15354/menu-multiple-trigger branch from 078951d to f581ea3 Compare February 1, 2020 16:20
@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
@crisbeto
Copy link
Member Author

No longer needed after #30163.

@crisbeto crisbeto closed this Dec 14, 2024
@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 Jan 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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