Add preference to mention reblogger when replying to a reblogged status

Tusker/Preferences/Preferences.swift

@@ -44,8 +44,9 @@ class Preferences: Codable, ObservableObject {
         
         self.defaultPostVisibility = try container.decode(Status.Visibility.self, forKey: .defaultPostVisibility)
         self.automaticallySaveDrafts = try container.decode(Bool.self, forKey: .automaticallySaveDrafts)
-        self.contentWarningCopyMode = try container.decode(ContentWarningCopyMode.self, forKey: .contentWarningCopyMode)
         self.requireAttachmentDescriptions = try container.decode(Bool.self, forKey: .requireAttachmentDescriptions)
+        self.contentWarningCopyMode = try container.decode(ContentWarningCopyMode.self, forKey: .contentWarningCopyMode)
+        self.mentionReblogger = try container.decode(Bool.self, forKey: .mentionReblogger)
         self.blurAllMedia = try container.decode(Bool.self, forKey: .blurAllMedia)
         self.openLinksInApps = try container.decode(Bool.self, forKey: .openLinksInApps)
         self.useInAppSafari = try container.decode(Bool.self, forKey: .useInAppSafari)
@@ -68,8 +69,9 @@ class Preferences: Codable, ObservableObject {
         
         try container.encode(defaultPostVisibility, forKey: .defaultPostVisibility)
         try container.encode(automaticallySaveDrafts, forKey: .automaticallySaveDrafts)
-        try container.encode(contentWarningCopyMode, forKey: .contentWarningCopyMode)
         try container.encode(requireAttachmentDescriptions, forKey: .requireAttachmentDescriptions)
+        try container.encode(contentWarningCopyMode, forKey: .contentWarningCopyMode)
+        try container.encode(mentionReblogger, forKey: .mentionReblogger)
         try container.encode(blurAllMedia, forKey: .blurAllMedia)
         try container.encode(openLinksInApps, forKey: .openLinksInApps)
         try container.encode(useInAppSafari, forKey: .useInAppSafari)
@@ -91,8 +93,9 @@ class Preferences: Codable, ObservableObject {
     // MARK: - Behavior
     @Published var defaultPostVisibility = Status.Visibility.public
     @Published var automaticallySaveDrafts = true
-    @Published var contentWarningCopyMode = ContentWarningCopyMode.asIs
     @Published var requireAttachmentDescriptions = false
+    @Published var contentWarningCopyMode = ContentWarningCopyMode.asIs
+    @Published var mentionReblogger = false
     @Published var blurAllMedia = false
     @Published var openLinksInApps = true
     @Published var useInAppSafari = true
@@ -114,8 +117,9 @@ class Preferences: Codable, ObservableObject {
         
         case defaultPostVisibility
         case automaticallySaveDrafts
-        case contentWarningCopyMode
         case requireAttachmentDescriptions
+        case contentWarningCopyMode
+        case mentionReblogger
         case blurAllMedia
         case openLinksInApps
         case useInAppSafari

Tusker/Screens/Compose/ComposeViewController.swift

@@ -13,7 +13,7 @@ import Intents
 class ComposeViewController: UIViewController {
     
     var inReplyToID: String?
-    var accountsToMention: [String]
+    var accountsToMention = [String]()
     var initialText: String?
     var contentWarningEnabled = false {
         didSet {
@@ -74,11 +74,12 @@ class ComposeViewController: UIViewController {
         self.inReplyToID = inReplyToID
         if let inReplyToID = inReplyToID, let inReplyTo = MastodonCache.status(for: inReplyToID) {
             accountsToMention = [inReplyTo.account.acct] + inReplyTo.mentions.map { $0.acct }
-        } else if let mentioningAcct = mentioningAcct {
-            accountsToMention = [mentioningAcct]
         } else {
             accountsToMention = []
         }
+        if let mentioningAcct = mentioningAcct {
+            accountsToMention.append(mentioningAcct)
+        }
         if let ownAccount = MastodonController.account {
             accountsToMention.removeAll(where: { acct in ownAccount.acct == acct })
         }

Tusker/Screens/Preferences/BehaviorPrefsView.swift

@@ -13,14 +13,15 @@ struct BehaviorPrefsView: View {
 
     var body: some View {
         List {
-            section1
-            section2
-            section3
+            composingSection
+            replyingSection
+            readingSection
+            linksSection
         }.listStyle(GroupedListStyle())
             .navigationBarTitle(Text("Behavior"))
     }
     
-    var section1: some View {
+    var composingSection: some View {
         Section(header: Text("COMPOSING")) {
             Picker(selection: $preferences.defaultPostVisibility, label: Text("Default Post Visibility")) {
                 ForEach(Status.Visibility.allCases, id: \.self) { visibility in
@@ -35,18 +36,26 @@ struct BehaviorPrefsView: View {
             Toggle(isOn: $preferences.automaticallySaveDrafts) {
                 Text("Automatically Save Drafts")
             }
-            Picker(selection: $preferences.contentWarningCopyMode, label: Text("Content Warning Copy Style")) {
-                Text("As-is").tag(ContentWarningCopyMode.asIs)
-                Text("Prepend 're: '").tag(ContentWarningCopyMode.prependRe)
-                Text("Don't copy").tag(ContentWarningCopyMode.doNotCopy)
-            }
             Toggle(isOn: $preferences.requireAttachmentDescriptions) {
                 Text("Require Attachment Descriptions")
             }
         }
     }
     
-    var section2: some View {
+    var replyingSection: some View {
+        Section(header: Text("REPLYING")) {
+            Picker(selection: $preferences.contentWarningCopyMode, label: Text("Content Warning Copy Style")) {
+                Text("As-is").tag(ContentWarningCopyMode.asIs)
+                Text("Prepend 're: '").tag(ContentWarningCopyMode.prependRe)
+                Text("Don't copy").tag(ContentWarningCopyMode.doNotCopy)
+            }
+            Toggle(isOn: $preferences.mentionReblogger) {
+                Text("Mention Reblogger")
+            }
+        }
+    }
+    
+    var readingSection: some View {
         Section(header: Text("READING")) {
             Toggle(isOn: $preferences.blurAllMedia) {
                 Text("Blur All Media")
@@ -54,7 +63,7 @@ struct BehaviorPrefsView: View {
         }
     }
     
-    var section3: some View {
+    var linksSection: some View {
         Section(header: Text("LINKS")) {
             Toggle(isOn: $preferences.openLinksInApps) {
                 Text("Open Links in Apps")

Tusker/TuskerNavigationDelegate.swift

@@ -32,6 +32,8 @@ protocol TuskerNavigationDelegate {
     
     func reply(to statusID: String)
     
+    func reply(to statusID: String, mentioningAcct: String?)
+    
     func largeImage(_ image: UIImage, description: String?, sourceView: UIImageView) -> LargeImageViewController
     
     func largeImage(gifData: Data, description: String?, sourceView: UIImageView) -> LargeImageViewController
@@ -125,15 +127,19 @@ extension TuskerNavigationDelegate where Self: UIViewController {
         compose(mentioning: nil)
     }
     
-    func compose(mentioning: String? = nil) {
-        let compose = ComposeViewController( mentioningAcct: mentioning)
+    func compose(mentioning: String?) {
+        let compose = ComposeViewController(mentioningAcct: mentioning)
         let vc = UINavigationController(rootViewController: compose)
         vc.presentationController?.delegate = compose
         present(vc, animated: true)
     }
     
     func reply(to statusID: String) {
-        let compose = ComposeViewController(inReplyTo: statusID)
+        reply(to: statusID, mentioningAcct: nil)
+    }
+    
+    func reply(to statusID: String, mentioningAcct: String?) {
+        let compose = ComposeViewController(inReplyTo: statusID, mentioningAcct: mentioningAcct)
         let vc = UINavigationController(rootViewController: compose)
         vc.presentationController?.delegate = compose
         present(vc, animated: true)

Tusker/Views/Status/TimelineStatusTableViewCell.swift

@@ -112,6 +112,16 @@ class TimelineStatusTableViewCell: BaseStatusTableViewCell {
         }
     }
     
+    func reply() {
+        if Preferences.shared.mentionReblogger,
+            let rebloggerID = rebloggerID,
+            let rebloggerAccount = MastodonCache.account(for: rebloggerID) {
+            delegate?.reply(to: statusID, mentioningAcct: rebloggerAccount.acct)
+        } else {
+            delegate?.reply(to: statusID)
+        }
+    }
+    
     override func prepareForReuse() {
         super.prepareForReuse()
         updateTimestampWorkItem?.cancel()
@@ -124,6 +134,10 @@ class TimelineStatusTableViewCell: BaseStatusTableViewCell {
         delegate?.selected(account: rebloggerID)
     }
     
+    override func replyPressed() {
+        reply()
+    }
+
     override func getStatusCellPreviewProviders(for location: CGPoint, sourceViewController: UIViewController) -> BaseStatusTableViewCell.PreviewProviders? {
         return (
             content: { ConversationTableViewController(for: self.statusID, state: self.statusState.copy()) },
@@ -205,7 +219,7 @@ extension TimelineStatusTableViewCell: TableViewSwipeActionProvider {
     func trailingSwipeActionsConfiguration() -> UISwipeActionsConfiguration? {
         let reply = UIContextualAction(style: .normal, title: "Reply") { (action, view, completion) in
             completion(true)
-            self.delegate?.reply(to: self.statusID)
+            self.reply()
         }
         reply.image = UIImage(systemName: "arrowshape.turn.up.left.fill")
         reply.backgroundColor = tintColor