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

An item should remain highlighted as long as its associated submenu is being displayed #387

Closed
xty opened this issue Apr 11, 2016 · 3 comments
Labels

Comments

@xty
Copy link
Contributor

xty commented Apr 11, 2016

As can be observed from the screenshot below, while the submenu is still being displayed, the menu item turtle 32 becomes unhighlighted. I suspect this is because an contextmenu:blur event was triggered on turtle 32 when the mouse moved outside. This is not the normal behaviour for context menus and thus should be fixed.

screenshot from 2016-04-11 12-02-23

Steps to reproduce this behaviour:

  1. Create and trigger a context menu with a submenu.
  2. Expand the submenu by hovering the mouse over the menu item with which the submenu is associated.
  3. Move the mouse out of the menu without moving into the submenu or any other menu items.
@bbrala bbrala added the Bug label Apr 11, 2016
@bbrala
Copy link
Member

bbrala commented Apr 11, 2016

I see what you mean. if the menu is a submenu it shouldn't unhighlight until submenu is closed. Thanks for the report. Can you tell me what version you use?

@xty
Copy link
Contributor Author

xty commented Apr 11, 2016

@bbrala I'm using v2.1.0. Note the behaviour in question can also be observed in the submenu demo.

screenshot from 2016-04-11 21-03-18
This is how it should always look like.

screenshot from 2016-04-11 21-03-39
However, this is how it looks like if the steps above are followed.

@bbrala bbrala closed this as completed in bbbf33c Dec 15, 2016
@bbrala
Copy link
Member

bbrala commented Dec 15, 2016

I've tweaked positioning a bit to make this harder. Don't really want to do extra checks in the menu itself for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants