Updated selected row color

Author: mikoldin123

Session/Conversations/Context Menu/ContextMenuVC+Action.swift

@@ -340,29 +340,23 @@ extension ContextMenuVC {
         delegate: ContextMenuActionDelegate?,
         using dependencies: Dependencies
     ) -> [Action]? {
-        let canCopy: Bool = (
-            cellViewModel.cellType == .textOnlyMessage || (
-                (
-                    cellViewModel.cellType == .genericAttachment ||
-                    cellViewModel.cellType == .mediaMessage
-                ) &&
-                (cellViewModel.attachments ?? []).count == 1 &&
-                (cellViewModel.attachments ?? []).first?.isVisualMedia == true &&
-                (cellViewModel.attachments ?? []).first?.isValid == true && (
-                    (cellViewModel.attachments ?? []).first?.state == .downloaded ||
-                    (cellViewModel.attachments ?? []).first?.state == .uploaded
-                )
-            )
-        )
-        
         let canDelete: Bool = (MessageViewModel.DeletionBehaviours.deletionActions(
             for: [cellViewModel],
             with: threadViewModel,
             using: dependencies
         ) != nil)
+        
+        var showDelete: Bool {
+            cellViewModel.attachments != nil && canDelete
+        }
+        
+        var showCopy: Bool {
+            cellViewModel.cellType == .textOnlyMessage
+        }
+        
         let generatedActions: [Action] = [
-            (canCopy ? Action.copy(cellViewModel, delegate) : nil),
-            (canDelete ? Action.delete(cellViewModel, delegate) : nil),
+            (showCopy ? Action.copy(cellViewModel, delegate) : nil),
+            (showDelete ? Action.delete(cellViewModel, delegate) : nil),
             Action.info(cellViewModel, delegate)
         ]
         .compactMap { $0 }

Session/Conversations/ConversationVC+Interaction.swift

@@ -3526,6 +3526,14 @@ extension ConversationVC {
 // MARK: - Multiple selection handling
 extension ConversationVC {
     func shouldHandleMessageSelection(for message: MessageViewModel, in cell: UITableViewCell) {
+        guard message.variant == .standardIncoming || (
+            message.variant == .standardOutgoing &&
+            message.state != .failed &&
+            message.state != .sending
+        ) else {
+            return
+        }
+        
         if let selectedIndex = selectedMessages.firstIndex(where: { $0 == message }) {
             selectedMessages.remove(at: selectedIndex)
         } else {
@@ -3575,4 +3583,19 @@ extension ConversationVC: SelectionManagerDelegate {
             inset: Values.largeSpacing + (inputAccessoryView?.frame.height ?? 0)
         )
     }
+    
+    func showInfo(for message: MessageViewModel, withSender sender: UIBarButtonItem) {
+        /*guard
+            let actions = ContextMenuVC.navigationActions(
+                for: message,
+                in: viewModel.threadData,
+                delegate: self,
+                using: viewModel.dependencies
+            )
+        else { return }*/
+        
+        // TODO: - Show context menu below navigationbar for now navigate to info page
+        info(message)
+        resetSelection()
+    }
 }

Session/Conversations/Message Cells/MessageCell.swift

@@ -46,7 +46,7 @@ public class MessageCell: UITableViewCell {
     }
 
     func setSelectedState(_ selected: Bool) {
-        contentView.backgroundColor = selected ? .orange : .clear
+        themeBackgroundColor = selected ? .backgroundSecondary : .clear
     }
 
     func setUpGestureRecognizers() {

Session/Conversations/Selection/MessageSelectionManager.swift

@@ -8,6 +8,7 @@ protocol SelectionManagerDelegate: ContextMenuActionDelegate {
     func willDeleteMessages(_ messages: [MessageViewModel], completion: @escaping () -> Void)
     func shouldResetSelectionState()
     func shouldShowCopyToast()
+    func showInfo(for message: MessageViewModel, withSender sender: UIBarButtonItem)
 
     var selectedMessages: Set<MessageViewModel> { get }
 }
@@ -140,13 +141,15 @@ class MessageSelectionManager: NSObject {
     }
 
     @objc
-    func moreOptions() {
+    func moreOptions(_ sender: UIBarButtonItem) {
         guard let selectedMessage = selectedMessages.first else {
             return
         }
-        // TODO: Show drop down instead
-        delegate?.info(selectedMessage)
 
-        delegate?.shouldResetSelectionState()
+        // TODO: Show drop down instead
+        delegate?.showInfo(for: selectedMessage, withSender: sender)
+//        delegate?.info(selectedMessage)
+//        
+//        delegate?.shouldResetSelectionState()
     }
 }