Context menu flickering(may be it re-render) after modify message data

Hi, i just found this problem. it’s REALLY HARD TO KNOW.

you can see the video here, as I just add a reaction to view:

https://user-images.githubusercontent.com/8202437/174272591-80fdb8cc-c6eb-4c04-baa2-15015bcbfee8.mov

Here is the main layout for view:

<ContextMenuView>
   <Bubble/>
  <ReactionView/>
</ContextMenuView>

Inside onPressMenuItem, i just update message:

const message = props.currentMessage
                            const reactions = message.reactions
                            reactions.push({
                                'id': '1',
                                'type': 200,
                                'date': new Date().getTime(),
                                'by': currentUser.id
                            })
                            message.reactions = reactions
                            handleChoosenMessageReaction(message)

const handleChoosenMessageReaction = React.useCallback((message) => {
        const newMessages = [...messages]
        const index = newMessages.findIndex(mes => mes._id === message._id)
        console.log('index', index)
        if(index > -1){
            newMessages[index].reactions = [...message.reactions]
            console.log('handleChoosenMessageReaction', message.reactions)
            setMessage(newMessages)
        }
    })

I tried to use https://github.com/mpiannucci/react-native-context-menu-view but it’s does not make problem, But I want to use your lib because it’s better

Please helppp

[dominicstop]

Hey, this has been fixed in version 1.8.1 ✨

I also added shouldWaitForMenuToHideBeforeFiringOnPressMenuItem prop to ContextMenuView if you want onPressMenu to fire ASAP.

Here’s an excerpt from the docs:

🔤 shouldWaitForMenuToHideBeforeFiringOnPressMenuItem ⚛️ boolean ✳️ Default: true
If set to true, the onPressMenuItem event will be triggered after the context menu has been hidden (i.e. after onMenuDidHide event is triggered).
Set this to false if you want onPressMenuItem to trigger as soon as an item has been pressed in the context menu.
📝 Note: Layout updates while the context menu is transitioning from it’s open to hidden state might cause layout bugs (e.g. https://github.com/dominicstop/react-native-ios-context-menu/issues/43).

Thanks for filing an issue 😌

[dominicstop]

@fukemy Hey, I’ve added Test02Screen to reproduce your bug.

https://user-images.githubusercontent.com/18517029/174292418-a660a28a-86a9-43a3-b33a-51f35ef43160.mov

As a temp. solution, I’m going to update onPressMenuItem so that it fires after the context menu is hidden first

Once i finish releasing the patch, I’ll close this issue 😌