Don't offer to send reactions for video-invitations and others (#2084)

Thanks to @Simon-Laux for the hints
deltachat · Feb 26, 2024 · c204626 · c204626
1 parent 48d00b8
commit c204626
Showing 1 changed file with 17 additions and 4 deletions.
diff --git a/deltachat-ios/Chat/ChatViewController.swift b/deltachat-ios/Chat/ChatViewController.swift
@@ -205,17 +205,30 @@ class ChatViewController: UITableViewController, UITableViewDropDelegate {
     }
 
     private func contextMenu(for indexPath: IndexPath) -> ContextMenuProvider {
+
+        let menuItems: [ContextMenuProvider.ContextMenuItem]
+
         if #available(iOS 13.0, *) {
             if dcChat.canSend {
-                return ContextMenuProvider(menu: [reactionsMenu(indexPath: indexPath), replyItem, replyPrivatelyItem, forwardItem, infoItem, copyItem, deleteItem, selectMoreItem])
+                let messageId = messageIds[indexPath.row]
+                let message = dcContext.getMessage(id: messageId)
+                let showReaction = message.isInfo == false && message.isSetupMessage == false && message.viewtype != nil && message.viewtype != .video
+
+                if showReaction {
+                    menuItems = [reactionsMenu(indexPath: indexPath), replyItem, replyPrivatelyItem, forwardItem, infoItem, copyItem, deleteItem, selectMoreItem]
+                } else {
+                    menuItems = [replyItem, replyPrivatelyItem, forwardItem, infoItem, copyItem, deleteItem, selectMoreItem]
+                }
             } else {
-                return ContextMenuProvider(menu: [replyPrivatelyItem, forwardItem, infoItem, copyItem, deleteItem])
+                menuItems = [replyPrivatelyItem, forwardItem, infoItem, copyItem, deleteItem]
             }
         } else if dcChat.canSend { // skips some options on iOS <13 because of limited horizontal space (reply is still available by swiping)
-            return ContextMenuProvider(menu: [forwardItem, infoItem, copyItem, deleteItem, selectMoreItem])
+            menuItems = [forwardItem, infoItem, copyItem, deleteItem, selectMoreItem]
         } else {
-            return ContextMenuProvider(menu: [forwardItem, infoItem, copyItem, deleteItem])
+            menuItems = [forwardItem, infoItem, copyItem, deleteItem]
         }
+
+        return ContextMenuProvider(menu: menuItems)
     }
 
     private lazy var copyItem: ContextMenuProvider.ContextMenuItem = {