[SOLVED] Context menu crashes app when wrapping a navigation trigger about zeego

[nandorojo]

ah that’s not good! i’m working through some other issues with the menu but i’ll try to address this if the reason becomes apparent. it’s hard coordinating with native view controllers for these edge cases

[nandorojo]

@peterferguson this is getting fixed upstream in the context menu v2:

dominicstop/react-native-ios-context-menu#77 (comment)

[sidorchukandrew]

@peterferguson There's a new version of react-native-ios-context-menu out.

[nandorojo]

There are some fixes underway for v2

[peterferguson]

Found this on issue that solves the problem

dominicstop/react-native-ios-context-menu#9 (comment)

Maybe for zeego is it just better to have this documented somewhere?

A FAQ or common pitfalls section?

I will leave it up to you whether to close or not (might be worth leaving open if you are open to the docs changes?)

[nandorojo]

Oh yeah that does make sense. I wonder if we should make it the default behavior or not. Or I also wonder if there is a way to fix this upstream entirely.

[peterferguson]

Seems like it could be simple enough to check if the child passed to the trigger has onLongPress set and if not override it.

Also a reasonably small change that (on first glance) shouldn't have many repercussions in other trigger use-cases.

[nandorojo]

The issue is, the child of the context menu could be anywhere (as in multiple children down).

So if a nested child is the pressable, it won't get auto-fixed.

[sidorchukandrew]

This seems to still be happening on the latest version of zeego (1.7.2) and react-native-ios-context-menu (2.3.0). Is it possible that zeego needs to set shouldPreventLongPressGestureFromPropagating on the ContextMenuView? It looks like it's a new prop (I don't know much Swift but it also looks like the default value is getting set to true)

[nandorojo]

Hmm good to know

[nandorojo]

@sidorchukandrew you can now pass __unsafeIosProps to Zeego's ContextMenu to test this. It'll forward props down

[sidorchukandrew]

I'll get that tested shortly!

[sidorchukandrew]

@nandorojo Is there a new release I should install?

[peterferguson]

Just tried this

<ContextMenu.Root __unsafeIosProps={{ shouldPreventLongPressGestureFromPropagating: true }}>

but is still crashing for me on [email protected] & [email protected]