From 53ec8cc57f3dd4567e436063ae2c77fec1e1eff9 Mon Sep 17 00:00:00 2001 From: "Krishna (via MelvinBot)" Date: Thu, 12 Mar 2026 09:48:06 +0000 Subject: [PATCH] Fix context menu not opening on long press of attachments The context provider value in PureReportActionItem used JavaScript shorthand property notation for shouldDisplayContextMenuValue, which created a property named 'shouldDisplayContextMenuValue' instead of the expected 'shouldDisplayContextMenu'. All consumers (ImageRenderer, VideoPlayerThumbnail, BaseAnchorForAttachmentsOnly, etc.) destructure 'shouldDisplayContextMenu' from the context and got undefined, causing their onLongPress handlers to return early. Co-authored-by: Krishna --- src/pages/inbox/report/PureReportActionItem.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/pages/inbox/report/PureReportActionItem.tsx b/src/pages/inbox/report/PureReportActionItem.tsx index 85fd24e9ccb1..54f9f9cf8070 100644 --- a/src/pages/inbox/report/PureReportActionItem.tsx +++ b/src/pages/inbox/report/PureReportActionItem.tsx @@ -831,7 +831,7 @@ function PureReportActionItem({ action, transactionThreadReport, isDisabled: false, - shouldDisplayContextMenuValue, + shouldDisplayContextMenu: shouldDisplayContextMenuValue, }), [report, action, transactionThreadReport, shouldDisplayContextMenuValue, isReportArchived], );