material icon indicating copy to clipboard operation
material copied to clipboard

menu: hover issue when moving quickly in and out of sub menus

Open kvkv opened this issue 8 years ago • 5 comments

@dremie Actual Behavior:

  • What is the issue? * mouse hover on menu , it is working fine. If we go to another menu (immediately or fast) hover sometimes previous submenu is not closed
  • What is the expected behavior? If we go to another menu , previous submenu should be closed

CodePen (or steps to reproduce the issue): *

AngularJS Versions: *

  • AngularJS Version: 1.3
  • AngularJS Material Version:1.1.5

Additional Information:

  • Browser Type: * All browsers
  • Browser Version: * All versions
  • OS: * All OS
  • Stack Traces: submenu-hover-codeopen

kvkv avatar Dec 11 '17 08:12 kvkv

Are you seeing this previous menu not closing temporarily or permanently? I do see some overlap and momentary animation issues, but after less than 1s it stabilizes and the previous menu is closed. Are you seeing the previous menu stay open for more than 1s?

Splaktar avatar Dec 13 '17 08:12 Splaktar

Same issue is reproducible in Angular Material 1.1.4 The previous menu will close with the current menu.

JamesPJ avatar Dec 19 '17 07:12 JamesPJ

@Splaktar

Previous menu stay open more than 1s( not closing, current and previous menu both are visible). Previous menu is closing with the current menu. It is not coming always but reproduce easily.

I gave the codepen example. Please check the issue on the coedpen example. We can reproduce the issue easily. Go fast one menu to other menu. after some time it should be reprodue.

kvkv avatar Dec 26 '17 12:12 kvkv

I've fixed the issue here https://github.com/JamesPJ/material/commit/e31ea91fe1f16a0d5f3f96650e2ce6361655a64a#diff-0b064159db8aac9de4c48b1927b24731

And the updated codepen https://codepen.io/Skybuzz/pen/ZvyEKe

Update: Tested

JamesPJ avatar Jan 01 '18 10:01 JamesPJ

@JamesPJ I am not able to run the CodePen you posted as the links to your js/css files don't work. However, I would love to check out your changes if you could post a PR!

Splaktar avatar Mar 27 '18 08:03 Splaktar