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

bug(menu): menu is triggering custom animation of a custom component used inside it #27301

Open
1 task
iKrishnaSahu opened this issue Jun 14, 2023 · 3 comments
Open
1 task
Labels
animation This issue is related to Angular animations or CSS animations area: material/menu P4 A relatively minor issue that is not relevant to core functions

Comments

@iKrishnaSahu
Copy link

Is this a regression?

  • Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was

No response

Description

bug(menu): menu is triggering custom animation of a custom component used inside it

StackBlitz link - https://stackblitz.com/edit/8ve6wn?file=src%2Fapp%2Fmenu-icons-example.html

chrome-capture-2023-5-14

Reproduction

Steps to reproduce:

  1. Visit this stackblitz - https://stackblitz.com/edit/8ve6wn?file=src%2Fapp%2Fmenu-icons-example.html
  2. Once the UI loads, click on vertical ellipsis button
  3. Click outside the popup. For a fraction of section, you will see the animation of demo-statusbar has been triggered and content is visible
  4. Open the menu again and check extra content from demo-statusbar is visible

Expected Behavior

Closing menu should not trigger any custom component's custom animation

Actual Behavior

Closing menu is triggering custom component's custom animation

Environment

  • Angular: 15
  • CDK/Material: 15
  • Browser(s): Chrome
  • Operating System (e.g. Windows, macOS, Ubuntu): Windows
@iKrishnaSahu iKrishnaSahu added the needs triage This issue needs to be triaged by the team label Jun 14, 2023
@iKrishnaSahu
Copy link
Author

I am unable to reproduce this in Angular 16 as stackblitz is down for v16 - #27300

@iKrishnaSahu
Copy link
Author

@mmalerba
Copy link
Contributor

Confirmed the issue, here's a less noisy version of the stackblitz that can still repro: https://stackblitz.com/edit/8ve6wn-pvbwzz (only a single status bar instance, and removed the second animation)

@mmalerba mmalerba added P4 A relatively minor issue that is not relevant to core functions animation This issue is related to Angular animations or CSS animations area: material/menu and removed needs triage This issue needs to be triaged by the team labels Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
animation This issue is related to Angular animations or CSS animations area: material/menu P4 A relatively minor issue that is not relevant to core functions
Projects
None yet
Development

No branches or pull requests

2 participants