Menu list items are pressable after close

[SohelIslamImran]

Describe the bug
When you first time open manu, the area of where the menu list items were opened, are pressable. Menu is hidden, But still can press the menu items. Can't understand what is the problem .. Please fix.

Package versions
"react": "17.0.2",
"react-native": "0.68.2",
"react-native-reanimated": "~2.8.0",
"react-native-hold-menu": "^0.1.5",
"expo": "^45.0.6",

[SohelIslamImran]

@enesozturk

[SohelIslamImran]

I think you need to minus (-) zindex of menu, when close/inactive

[SohelIslamImran]

Another problem found @enesozturk , items props on HoldItem component are not changing, it kept memorized. So it is causing problems with multiple HoldItem on multiple screens. When I open a hold menu and then go to another screen and open another hold menu, it keeps the previous items, not showing this new menu items.

[SohelIslamImran]

Unfortunately, this package is not working as expected and as shown in the documentation, examples and gif. So sad☹️.

[enesozturk]

This is indeed unexpected and I cannot reproduce the same issue you recorded. Do you get the same with all the use cases? Which version are you using?

items props on HoldItem component are not changing

This was one of the old issues and I remember it's solved. When I run the example on the same device (It's Expo, so would be easy to test), I cannot see the same issue between the examples.

[SohelIslamImran]

Alright. But let's quick fix the first and main issue. (The Menu list items pressable issue on android) @enesozturk

[SohelIslamImran]

This is indeed unexpected and I cannot reproduce the same issue you recorded. Do you get the same with all the use cases? Which version are you using?

items props on HoldItem component are not changing

This was one of the old issues and I remember it's solved. When I run the example on the same device (It's Expo, so would be easy to test), I cannot see the same issue between the examples.

Actually, I found the exact issue. The deepEqual in react-native-hold-menu\src\components\menu\MenuList.tsx are returning true, even they are not equal. Please check and fix the deepEqual. @enesozturk

Mainly, this condition is not working on React native. It always returns true even if functions are not equal.

[SohelIslamImran]

Please try to fix these two issues ASAP @enesozturk 🙏🙏!!

[SohelIslamImran]

In browser or node, function.toString() convert the function body to string. Like this

But in React Native, it always converts to this. So the equal (===) condition always returns true.

Is it clear? @enesozturk

[SohelIslamImran]

I found this issue facebook/hermes#114 and facebook/hermes#612 (comment)

[SohelIslamImran]

Bump. Waiting

[SohelIslamImran]

In browser or node, function.toString() convert the function body to string. Like this

But in React Native, it always converts to this. So the equal (===) condition always returns true.

Is it clear? @enesozturk

I console logged normal function. It is showing the function body. The reason of the above issue is you used useSharedValue for store items. I think you need to use react state or context. @enesozturk

[SohelIslamImran]

@enesozturk Can you please fix these issues ASAP? My app is buggy for this package now. Or instead, I should find another package?

[kesha-antonov]

Hi
I encountered this issue also on Android 11
Area where menu was remains tappable

I use latest version of the lib

UPD. Created PR for this

#115