fix: re-open context-menu on closing contextmenu event

[tomivirkki]

Description

Dispatch a new contextmenu event over the target in case the context-menu gets closed by a contextmenu event. This enables opening a new context menu without closing the previous one first.

Kapture.2024-06-11.at.13.38.34.mp4

Fixes #1135

Type of change

Bugfix

[DiegoCardoso]

LGTM

The only question I have is whether this can be considered a breaking behavioral change since it required two right clicks before opening a new context menu.

[tomivirkki]

The only question I have is whether this can be considered a breaking behavioral change since it required two right clicks before opening a new context menu.

That's a valid point. The new behavior is aligned with how native context menus work in general (in Finder etc) so it might also be expected by users. To be on the safe side, we could at least skip cherry-picking this to earlier minors (and only target 24.5). @rolfsmeds @yuriy-fix wdyt?

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[tomivirkki]

We discussed this internally and agreed to merge this to 24.5 as a minor behavioral breaking change:

• Before: A right click outside an open context menu would close the context menu. A second right click would open a new one.
• After: A right click outside an open context menu would close the context menu and open another one to the coordinates of the click.

[vaadin-bot]

This ticket/PR has been released with Vaadin 24.5.0.alpha4 and is also targeting the upcoming stable 24.5.0 version.