ShareExtension/Info.plist

@@ -8,8 +8,6 @@
 		<dict>
 			<key>NSExtensionActivationRule</key>
 			<dict>
-				<key>NSExtensionActivationSupportsText</key>
-				<true/>
 				<key>NSExtensionActivationSupportsImageWithMaxCount</key>
 				<integer>4</integer>
 				<key>NSExtensionActivationSupportsMovieWithMaxCount</key>

ShareExtension/ShareViewController.swift

@@ -71,43 +71,36 @@ class ShareViewController: UIViewController {
     
     private func getDraftConfigurationFromExtensionContext() async -> (String, [DraftAttachment]) {
         guard let extensionContext,
-              let inputItem = (extensionContext.inputItems as? [NSExtensionItem])?.first else {
+              let inputItem = (extensionContext.inputItems as? [NSExtensionItem])?.first,
+              let itemProvider = inputItem.attachments?.first else {
             return ("", [])
         }
-        var text: String = ""
-        var url: URL?
-        var attachments: [DraftAttachment] = []
-        
-        for itemProvider in inputItem.attachments ?? [] {
-            if let attached: NSURL = await getObject(from: itemProvider) {
-                if url == nil {
-                    url = attached as URL
-                }
-            } else if let s: NSString = await getObject(from: itemProvider) {
-                if text.isEmpty {
-                    text = s as String
-                }
-            } else if let attachment: DraftAttachment = await getObject(from: itemProvider) {
-                attachments.append(attachment)
+        if let url: NSURL = await getObject(from: itemProvider) {
+            if let title = inputItem.attributedTitle ?? inputItem.attributedContentText {
+                return ("\n\n\(title.string)\n\(url.absoluteString ?? "")", [])
+            } else {
+                return ("\n\n\(url.absoluteString ?? "")", [])
             }
-        }
-        
-        if text.isEmpty,
-           let s = inputItem.attributedTitle ?? inputItem.attributedContentText {
-            text = s.string
-        }
-        
-        if let url {
-            if !text.isEmpty {
-                text += "\n"
+        } else if let text: NSString = await getObject(from: itemProvider) {
+            return ("\n\n\(text)", [])
+        } else if let attributedContent = inputItem.attributedContentText {
+            return ("\n\n\(attributedContent.string)", [])
+        } else {
+            let attachments = await withTaskGroup(of: DraftAttachment?.self, returning: [DraftAttachment].self) { group in
+                for provider in inputItem.attachments! {
+                    group.addTask { @MainActor in
+                        await self.getObject(from: provider)
+                    }
+                }
+                
+                return await group.reduce(into: [], { partialResult, result in
+                    if let result {
+                        partialResult.append(result)
+                    }
+                })
             }
-            text += url.absoluteString
+            return ("", attachments)
         }
-        
-        if !text.isEmpty {
-            text = "\n\n\(text)"
-        }
-        return (text, attachments)
     }
     
     private func getObject<T: NSItemProviderReading>(from itemProvider: NSItemProvider) async -> T? {

Tusker/ImageGrayscalifier.swift

@@ -9,7 +9,7 @@
 import UIKit
 
 struct ImageGrayscalifier {
-    static let queue = DispatchQueue(label: "ImageGrayscalifier", qos: .userInitiated)
+    static let queue = DispatchQueue(label: "ImageGrayscalifier", qos: .default)
     
     private static let context = CIContext()
     private static let cache = NSCache<NSURL, UIImage>()
@@ -24,17 +24,6 @@ struct ImageGrayscalifier {
         }
     }
     
-    static func convert(url: URL?, image: UIImage) -> UIImage? {
-        if let url,
-           let cached = cache.object(forKey: url as NSURL) {
-            return cached
-        }
-        guard let cgImage = image.cgImage else {
-            return nil
-        }
-        return doConvert(CIImage(cgImage: cgImage), url: url)
-    }
-    
     static func convert(url: URL?, data: Data) -> UIImage? {
         if let url = url,
            let cached = cache.object(forKey: url as NSURL) {

Tusker/Views/Attachments/AttachmentView.swift

@@ -75,7 +75,9 @@ class AttachmentView: GIFImageView {
         gifPlaybackModeChanged()
         
         if isGrayscale != Preferences.shared.grayscaleImages {
-            self.displayImage()
+            ImageGrayscalifier.queue.async {
+                self.displayImage()
+            }
         }
         
         if getBadges().isEmpty != Preferences.shared.showAttachmentBadges {
@@ -187,57 +189,51 @@ class AttachmentView: GIFImageView {
     
     func loadImage() {
         let attachmentURL = attachment.url
-        attachmentRequest = ImageCache.attachments.get(attachmentURL) { [weak self] (data, image) in
-            guard let self = self,
-                  self.attachment.url == attachmentURL else {
-                return
-            }
-            
+        attachmentRequest = ImageCache.attachments.get(attachmentURL) { [weak self] (data, _) in
+            guard let self = self, let data = data else { return }
             DispatchQueue.main.async {
                 self.attachmentRequest = nil
-                
-                if attachmentURL.pathExtension == "gif",
-                   let data {
-                    self.source = .gifData(attachmentURL, data, image)
+            }
+            if self.attachment.url.pathExtension == "gif" {
+                self.source = .gifData(attachmentURL, data)
+                let controller = GIFController(gifData: data)
+                DispatchQueue.main.async {
+                    controller.attach(to: self)
                     if self.autoplayGifs {
-                        let controller = GIFController(gifData: data)
-                        controller.attach(to: self)
                         controller.startAnimating()
-                    } else {
-                        self.displayImage()
                     }
-                } else if let image {
-                    self.source = .image(attachmentURL, image)
+                }
+                
+                if !self.autoplayGifs {
                     self.displayImage()
                 }
+                
+            } else {
+                self.source = .imageData(attachmentURL, data)
+                self.displayImage()
             }
         }
     }
     
     func loadVideo() {
         if let previewURL = self.attachment.previewURL {
-            attachmentRequest = ImageCache.attachments.get(previewURL, completion: { [weak self] (_, image) in
-                guard let self, let image else { return }
+            attachmentRequest = ImageCache.attachments.get(previewURL, completion: { [weak self] (data, _ )in
+                guard let self = self, let data = data else { return }
                 DispatchQueue.main.async {
                     self.attachmentRequest = nil
-                    self.source = .image(previewURL, image)
+                    self.source = .imageData(previewURL, data)
                     self.displayImage()
                 }
             })
         } else {
             let attachmentURL = self.attachment.url
-            AttachmentView.queue.async { [weak self] in
+            AttachmentView.queue.async {
                 let asset = AVURLAsset(url: attachmentURL)
                 let generator = AVAssetImageGenerator(asset: asset)
                 generator.appliesPreferredTrackTransform = true
                 guard let image = try? generator.copyCGImage(at: .zero, actualTime: nil) else { return }
-                UIImage(cgImage: image).prepareForDisplay { [weak self] image in
-                    DispatchQueue.main.async { [weak self] in
-                        guard let self, let image else { return }
-                        self.source = .image(attachmentURL, image)
-                        self.displayImage()
-                    }
-                }
+                self.source = .cgImage(attachmentURL, image)
+                self.displayImage()
             }
         }
         
@@ -280,10 +276,8 @@ class AttachmentView: GIFImageView {
             let generator = AVAssetImageGenerator(asset: asset)
             generator.appliesPreferredTrackTransform = true
             guard let image = try? generator.copyCGImage(at: .zero, actualTime: nil) else { return }
-            DispatchQueue.main.async {
-                self.source = .cgImage(attachmentURL, image)
-                self.displayImage()
-            }
+            self.source = .cgImage(attachmentURL, image)
+            self.displayImage()
         }
         
         let gifvView = GifvAttachmentView(asset: asset, gravity: .resizeAspectFill)
@@ -301,51 +295,33 @@ class AttachmentView: GIFImageView {
         ])
     }
     
-    @MainActor
     private func displayImage() {
         isGrayscale = Preferences.shared.grayscaleImages
         
+        let image: UIImage?
+        
         switch source {
         case nil:
-            self.image = nil
+            image = nil
             
-        case let .image(url, sourceImage):
+        case let .imageData(url, data), let .gifData(url, data):
             if isGrayscale {
-                ImageGrayscalifier.queue.async { [weak self] in
-                    let grayscale = ImageGrayscalifier.convert(url: url, image: sourceImage)
-                    DispatchQueue.main.async { [weak self] in
-                        self?.image = grayscale
-                    }
-                }
+                image = ImageGrayscalifier.convert(url: url, data: data)
             } else {
-                self.image = sourceImage
-            }
-            
-        case let .gifData(url, _, sourceImage):
-            if isGrayscale,
-               let sourceImage {
-                ImageGrayscalifier.queue.async { [weak self] in
-                    let grayscale = ImageGrayscalifier.convert(url: url, image: sourceImage)
-                    DispatchQueue.main.async { [weak self] in
-                        self?.image = grayscale
-                    }
-                }
-            } else {
-                self.image = sourceImage
+                image = UIImage(data: data)
             }
             
         case let .cgImage(url, cgImage):
             if isGrayscale {
-                ImageGrayscalifier.queue.async { [weak self] in
-                    let grayscale = ImageGrayscalifier.convert(url: url, cgImage: cgImage)
-                    DispatchQueue.main.async { [weak self] in
-                        self?.image = grayscale
-                    }
-                }
+                image = ImageGrayscalifier.convert(url: url, cgImage: cgImage)
             } else {
                 image = UIImage(cgImage: cgImage)
             }
         }
+        
+        DispatchQueue.main.async {
+            self.image = image
+        }
     }
     
     private func createBadgesView(_ badges: Badges) {
@@ -433,8 +409,8 @@ class AttachmentView: GIFImageView {
 
 fileprivate extension AttachmentView {
     enum Source {
-        case image(URL, UIImage)
-        case gifData(URL, Data, UIImage?)
+        case imageData(URL, Data)
+        case gifData(URL, Data)
         case cgImage(URL, CGImage)
     }