vaadin / vaadin-context-menu

The responsive Web Component for showing context dependent items for any element on the page. Part of the Vaadin components.
https://vaadin.com/components
Apache License 2.0
26 stars 10 forks source link

Fixed sub-menu gets old coordinate and some test-cases are fail after changing src code #260

Closed Harry-Ph closed 4 years ago

Harry-Ph commented 4 years ago

Fix:

  1. sub-menu gets old coordinate and is rendered. In old logic, sub-menu is rendered before received the right coordinate, so it gets old coordinate to use. I change the logic that gives the new coordinate and then renders it.
  2. Two test cases are failed and I fixed them due to new src code.
claassistantio commented 4 years ago

CLA assistant check
All committers have signed the CLA.