Skip to content

Close menu when selecting an item

Loic Huder requested to merge menu-close into main

Previously, the menu was staying open when selecting an entry (see screencast below) which is quite jarring compared to menus in other apps.

Screencast_from_23-05-2023_16_17_52

This MR adds a MenuContext to propagate the onClose handler to be able to close the menu when an item is selected. Prop drilling felt like a pain so I preferred to use a context.

Edited by Loic Huder

Merge request reports

Loading