Delay on ContextMenu onSelect

[wilkinsonj]

Hello,

I have a ContextMenu with a couple of ContextMenu.Items, and when I click one, the 'onSelect' event occurs around 1 second later. This is quite a noticeable lag from a user experience point of view.

Am I missing something to avoid this?

Also, the ContextMenu.Trigger with action 'press' does nothing on Press on iOS

Thanks

[nandorojo]

Never seen that happen. Can’t help without seeing all the code

[wilkinsonj]

The root onOpenWillChange fires immediately, and if I set shouldDismissMenuOnSelect to false, the onSelect works immediately.

Clearly, the delay is the closing down of the contextmenu, the event must be fired after that has happened. Will try to reproduce.

For info, it's a large-ish picture (200kb) within the trigger.

Will try to create a minimum reproduction

[nandorojo]

It may be a bug with the upstream iOS menu library in this case

[nandorojo]

I'm wondering if this prop could fix it:

shouldWaitForMenuToHideBeforeFiringOnPressMenuItem

 __unsafeIosProps={{
      shouldWaitForMenuToHideBeforeFiringOnPressMenuItem: false
    }}

[wilkinsonj]

Sounds likely! Will test it tonight / tomorrow in my code and feed back

[ahmadtech199]

action.press.not.working_resized.mp4

The ContextMenu.Trigger with action 'press' does nothing on Press on iOS
"zeego": "^1.9.1"

[nandorojo]

Please open a new issue, that's unrelated.

[Peek-A-Booo]

I'm wondering if this prop could fix it:

shouldWaitForMenuToHideBeforeFiringOnPressMenuItem

 __unsafeIosProps={{
      shouldWaitForMenuToHideBeforeFiringOnPressMenuItem: false
    }}

it works

[RamiroG8k]

I'm wondering if this prop could fix it:

shouldWaitForMenuToHideBeforeFiringOnPressMenuItem

 __unsafeIosProps={{
      shouldWaitForMenuToHideBeforeFiringOnPressMenuItem: false
    }}

Just for context to anyone with the same issue and the error with the types as mentioned in this issue, it goes in the Root

<ContextMenu.Root
    __unsafeIosProps={{
        shouldWaitForMenuToHideBeforeFiringOnPressMenuItem: false
    }}
>

[nandorojo]

Wondering if we should just set this field by default in Zeego. Any reason not to?