Open SohelIslamImran opened 2 years ago
@enesozturk
I think you need to minus (-) zindex of menu, when close/inactive
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.
Unfortunately, this package is not working as expected and as shown in the documentation, examples and gif. So sad☹️.
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.
Alright. But let's quick fix the first and main issue. (The Menu list items pressable issue on android) @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.
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.
Please try to fix these two issues ASAP @enesozturk 🙏🙏!!
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
Bump. Waiting
In browser or node,
function.toString()
convert the function body to string. Like thisBut in React Native, it always converts to this. So the equal (===) condition always returns true.
Is it clear? @enesozturk
useSharedValue
for store items. I think you need to use react state or context. @enesozturk@enesozturk Can you please fix these issues ASAP? My app is buggy for this package now. Or instead, I should find another package?
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
https://github.com/enesozturk/react-native-hold-menu/pull/115
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",