Packages/GalleryVC/Sources/GalleryVC/GalleryDismissAnimationController.swift

@@ -52,22 +52,15 @@ class GalleryDismissAnimationController: NSObject, UIViewControllerAnimatedTrans
             appliedSourceToDestTransform = false
         }
         
-        // Moving `to.view` to the container is necessary when the presenting VC (i.e., `to`)
-        // is in the window's root presentation.
-        // But it breaks when the gallery is presented from a sheet-presented VC--in which case
-        // `to.view` is already in the view hierarchy at this point; and adding it to the
-        // container causees it to be removed when the transition completes.
-        if to.view.superview == nil {
-            to.view.frame = container.bounds
-            container.addSubview(to.view)
-        }
-        
+        to.view.frame = container.bounds
         from.view.frame = container.bounds
-        container.addSubview(from.view)
 
         let content = itemViewController.takeContent()
         content.view.translatesAutoresizingMaskIntoConstraints = true
         content.view.layer.masksToBounds = true
+        
+        container.addSubview(to.view)
+        container.addSubview(from.view)
         container.addSubview(content.view)
         
         content.view.frame = destFrameInContainer

Tusker.xcodeproj/project.pbxproj

@@ -75,7 +75,6 @@
 		D620483423D3801D008A63EF /* LinkTextView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D620483323D3801D008A63EF /* LinkTextView.swift */; };
 		D620483623D38075008A63EF /* ContentTextView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D620483523D38075008A63EF /* ContentTextView.swift */; };
 		D620483823D38190008A63EF /* StatusContentTextView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D620483723D38190008A63EF /* StatusContentTextView.swift */; };
-		D6210D762C0B924F009BB569 /* RemoveProfileSuggestionService.swift in Sources */ = {isa = PBXBuildFile; fileRef = D6210D752C0B924F009BB569 /* RemoveProfileSuggestionService.swift */; };
 		D621733328F1D5ED004C7DB1 /* ReblogService.swift in Sources */ = {isa = PBXBuildFile; fileRef = D621733228F1D5ED004C7DB1 /* ReblogService.swift */; };
 		D62275A824F1CA2800B82A16 /* ComposeReplyContentView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D62275A724F1CA2800B82A16 /* ComposeReplyContentView.swift */; };
 		D623A53D2635F5590095BD04 /* StatusPollView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D623A53C2635F5590095BD04 /* StatusPollView.swift */; };
@@ -507,7 +506,6 @@
 		D620483323D3801D008A63EF /* LinkTextView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = LinkTextView.swift; sourceTree = "<group>"; };
 		D620483523D38075008A63EF /* ContentTextView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ContentTextView.swift; sourceTree = "<group>"; };
 		D620483723D38190008A63EF /* StatusContentTextView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = StatusContentTextView.swift; sourceTree = "<group>"; };
-		D6210D752C0B924F009BB569 /* RemoveProfileSuggestionService.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = RemoveProfileSuggestionService.swift; sourceTree = "<group>"; };
 		D621733228F1D5ED004C7DB1 /* ReblogService.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ReblogService.swift; sourceTree = "<group>"; };
 		D62275A724F1CA2800B82A16 /* ComposeReplyContentView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ComposeReplyContentView.swift; sourceTree = "<group>"; };
 		D623A53C2635F5590095BD04 /* StatusPollView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = StatusPollView.swift; sourceTree = "<group>"; };
@@ -1759,7 +1757,6 @@
 				D6DD8FFC298495A8002AD3FD /* LogoutService.swift */,
 				D6D79F252A0C8D2700AB2315 /* FetchStatusSourceService.swift */,
 				D630C3CB2BC5FD4600208903 /* GetAuthorizationTokenService.swift */,
-				D6210D752C0B924F009BB569 /* RemoveProfileSuggestionService.swift */,
 			);
 			path = API;
 			sourceTree = "<group>";
@@ -2365,7 +2362,6 @@
 				D698F46D2BD0B8310054DB14 /* AnnouncementsCollection.swift in Sources */,
 				D61F75AF293AF50C00C0B37F /* EditedFilter.swift in Sources */,
 				D691771929A7B8820054D7EF /* ProfileHeaderMovedOverlayView.swift in Sources */,
-				D6210D762C0B924F009BB569 /* RemoveProfileSuggestionService.swift in Sources */,
 				D61F75B9293C15A000C0B37F /* ZeroHeightCollectionViewCell.swift in Sources */,
 				D68232F72464F4FD00325FB8 /* ComposeDrawingViewController.swift in Sources */,
 				04586B4122B2FFB10021BD04 /* PreferencesView.swift in Sources */,

Tusker/API/RemoveProfileSuggestionService.swift

@@ -1,39 +0,0 @@
-//
-//  RemoveProfileSuggestionService.swift
-//  Tusker
-//
-//  Created by Shadowfacts on 6/1/24.
-//  Copyright © 2024 Shadowfacts. All rights reserved.
-//
-
-import Foundation
-import Pachyderm
-
-@MainActor
-class RemoveProfileSuggestionService {
-    private let accountID: String
-    private let mastodonController: MastodonController
-    private let presenter: any TuskerNavigationDelegate
-    private let completionHandler: @MainActor () -> Void
-    
-    init(accountID: String, mastodonController: MastodonController, presenter: any TuskerNavigationDelegate, completionHandler: @MainActor @escaping () -> Void) {
-        self.accountID = accountID
-        self.mastodonController = mastodonController
-        self.presenter = presenter
-        self.completionHandler = completionHandler
-    }
-    
-    func run() async {
-        let req = Suggestion.remove(accountID: accountID)
-        do {
-            _ = try await mastodonController.run(req)
-            completionHandler()
-        } catch {
-            let config = ToastConfiguration(from: error, with: "Error Removing Suggestion", in: presenter) { toast in
-                toast.dismissToast(animated: true)
-                await self.run()
-            }
-            self.presenter.showToast(configuration: config, animated: true)
-        }
-    }
-}

Tusker/Screens/Explore/SuggestedProfilesViewController.swift

@@ -184,19 +184,7 @@ extension SuggestedProfilesViewController: UICollectionViewDelegate {
         return UIContextMenuConfiguration {
             ProfileViewController(accountID: id, mastodonController: self.mastodonController)
         } actionProvider: { _ in
-            let dismiss = UIAction(title: "Remove Suggestion", image: UIImage(systemName: "trash"), attributes: .destructive) { [unowned self] _ in
-                let service = RemoveProfileSuggestionService(accountID: id, mastodonController: self.mastodonController, presenter: self) { [weak self] in
-                    guard let self else { return }
-                    var snapshot = self.dataSource.snapshot()
-                    // the source here doesn't matter, since it's ignored by the equatable and hashable impls
-                    snapshot.deleteItems([.account(id, .global)])
-                    self.dataSource.apply(snapshot, animatingDifferences: true)
-                }
-                Task {
-                    await service.run()
-                }
-            }
-            return UIMenu(children: [UIMenu(options: .displayInline, children: [dismiss])] + self.actionsForProfile(accountID: id, source: .view(cell)))
+            UIMenu(children: self.actionsForProfile(accountID: id, source: .view(cell)))
         }
     }
     

Tusker/Screens/Explore/TrendsViewController.swift

@@ -368,14 +368,20 @@ class TrendsViewController: UIViewController, CollectionViewController {
     
     @MainActor
     private func removeProfileSuggestion(accountID: String) async {
-        let service = RemoveProfileSuggestionService(accountID: accountID, mastodonController: mastodonController, presenter: self) { [weak self] in
-            guard let self else { return }
-            var snapshot = self.dataSource.snapshot()
+        let req = Suggestion.remove(accountID: accountID)
+        do {
+            _ = try await mastodonController.run(req)
+            var snapshot = dataSource.snapshot()
             // the source here doesn't matter, since it's ignored by the equatable and hashable impls
             snapshot.deleteItems([.account(accountID, .global)])
-            self.dataSource.apply(snapshot, animatingDifferences: true)
+            await apply(snapshot: snapshot)
+        } catch {
+            let config = ToastConfiguration(from: error, with: "Error Removing Suggestion", in: self) { [unowned self] toast in
+                toast.dismissToast(animated: true)
+                _ = await self.removeProfileSuggestion(accountID: accountID)
+            }
+            self.showToast(configuration: config, animated: true)
         }
-        await service.run()
     }
 }
 

Tusker/Screens/Main/AccountSwitchingContainerViewController.swift

@@ -123,7 +123,6 @@ class AccountSwitchingContainerViewController: UIViewController {
                     oldRoot.view.transform = CGAffineTransform(translationX: 0, y: -newInitialOffset).scaledBy(x: scale, y: scale)
                     newRoot.view.transform = .identity
                 } completion: { (_) in
-                    oldRoot.view.transform = .identity
                     oldRoot.additionalSafeAreaInsets = origAdditionalSafeAreaInsets
                     oldRoot.removeViewAndController()
                     newRoot.view.layer.masksToBounds = false

Tusker/Screens/Main/MainSplitViewController.swift

@@ -180,41 +180,26 @@ class MainSplitViewController: UISplitViewController {
     }
     
     @objc func handleSidebarCommandTimelines() {
-        if let previous = sidebar.selectedItem {
-            navigationStacks[previous] = secondaryNavController.viewControllers
-        }
         sidebar.select(item: .tab(.timelines), animated: false)
         select(item: .tab(.timelines))
     }
     
     @objc func handleSidebarCommandNotifications() {
-        if let previous = sidebar.selectedItem {
-            navigationStacks[previous] = secondaryNavController.viewControllers
-        }
         sidebar.select(item: .tab(.notifications), animated: false)
         select(item: .tab(.notifications))
     }
     
     @objc func handleSidebarCommandExplore() {
-        if let previous = sidebar.selectedItem {
-            navigationStacks[previous] = secondaryNavController.viewControllers
-        }
         sidebar.select(item: .tab(.explore), animated: false)
         select(item: .tab(.explore))
     }
     
     @objc func handleSidebarCommandBookmarks() {
-        if let previous = sidebar.selectedItem {
-            navigationStacks[previous] = secondaryNavController.viewControllers
-        }
         sidebar.select(item: .bookmarks, animated: false)
         select(item: .bookmarks)
     }
     
     @objc func handleSidebarCommandMyProfile() {
-        if let previous = sidebar.selectedItem {
-            navigationStacks[previous] = secondaryNavController.viewControllers
-        }
         sidebar.select(item: .tab(.myProfile), animated: false)
         select(item: .tab(.myProfile))
     }

Tusker/Screens/Utilities/MultiColumnNavigationController.swift

@@ -273,17 +273,12 @@ private class ColumnView: UIView {
     }
     
     private func installCloseBarButton(navigationItem: UINavigationItem) {
-        func makeItem() -> UIBarButtonItem {
-            let item = UIBarButtonItem(image: UIImage(systemName: "xmark"), style: .done, target: self, action: #selector(closeNavigationColumn))
-            item.accessibilityLabel = "Close Column"
-            return item
-        }
-        if let leftItems = navigationItem.leftBarButtonItems {
-            if !leftItems.contains(where: { $0.action == #selector(closeNavigationColumn) }) {
-                navigationItem.leftBarButtonItems!.insert(makeItem(), at: 0)
-            }
+        let item = UIBarButtonItem(image: UIImage(systemName: "xmark"), style: .done, target: self, action: #selector(closeNavigationColumn))
+        item.accessibilityLabel = "Close Column"
+        if navigationItem.leftBarButtonItems != nil {
+            navigationItem.leftBarButtonItems!.insert(item, at: 0)
         } else {
-            navigationItem.leftBarButtonItems = [makeItem()]
+            navigationItem.leftBarButtonItems = [item]
         }
     }
     

Tusker/Screens/Utilities/UIViewController+Children.swift

@@ -50,11 +50,8 @@ extension UIViewController {
     }
     
     func removeViewAndController() {
-        beginAppearanceTransition(false, animated: false)
         view.removeFromSuperview()
-        willMove(toParent: nil)
         removeFromParent()
-        endAppearanceTransition()
     }
 }
 

Tusker/Views/CachedImageView.swift

@@ -33,11 +33,6 @@ class CachedImageView: UIImageView {
         commonInit()
     }
     
-    deinit {
-        fetchTask?.cancel()
-        blurHashTask?.cancel()
-    }
-    
     private func commonInit() {
         NotificationCenter.default.addObserver(self, selector: #selector(preferencesChanged), name: .preferencesChanged, object: nil)
     }

Tusker/Views/Profile Header/ProfileHeaderView.swift

@@ -25,9 +25,9 @@ class ProfileHeaderView: UIView {
     weak var delegate: ProfileHeaderViewDelegate?
     var mastodonController: MastodonController! { delegate?.apiController }
 
-    @IBOutlet weak var headerImageView: CachedImageView!
+    @IBOutlet weak var headerImageView: UIImageView!
     @IBOutlet weak var avatarContainerView: UIView!
-    @IBOutlet weak var avatarImageView: CachedImageView!
+    @IBOutlet weak var avatarImageView: UIImageView!
     @IBOutlet weak var moreButton: ProfileHeaderButton!
     @IBOutlet weak var followButton: ProfileHeaderButton!
     @IBOutlet weak var displayNameLabel: AccountDisplayNameLabel!
@@ -44,6 +44,8 @@ class ProfileHeaderView: UIView {
     
     var accountID: String!
     
+    private var imagesTask: Task<Void, Never>?
+    
     private var isGrayscale = false
     private var followButtonMode = FollowButtonMode.follow {
         didSet {
@@ -54,6 +56,10 @@ class ProfileHeaderView: UIView {
         }
     }
     
+    deinit {
+        imagesTask?.cancel()
+    }
+    
     override func awakeFromNib() {
         super.awakeFromNib()
         
@@ -63,13 +69,11 @@ class ProfileHeaderView: UIView {
         avatarContainerView.layer.cornerCurve = .continuous
         // Set zPositions so the gallery presentation/dismissal animation looks correct.
         avatarContainerView.layer.zPosition = 2
-        avatarImageView.cache = .avatars
         avatarImageView.layer.masksToBounds = true
         avatarImageView.layer.cornerCurve = .continuous
         avatarImageView.addGestureRecognizer(UITapGestureRecognizer(target: self, action: #selector(avatarPressed)))
         avatarImageView.isUserInteractionEnabled = true
         
-        headerImageView.cache = .headers
         headerImageView.addGestureRecognizer(UITapGestureRecognizer(target: self, action: #selector(headerPressed)))
         headerImageView.isUserInteractionEnabled = true
         headerImageView.layer.zPosition = 1
@@ -134,11 +138,11 @@ class ProfileHeaderView: UIView {
         usernameLabel.text = "@\(account.acct)"
         lockImageView.isHidden = !account.locked
         
-        if let avatar = account.avatar {
-            avatarImageView.update(for: avatar)
-        }
-        if let header = account.header {
-            headerImageView.update(for: header)
+        imagesTask?.cancel()
+        let avatar = account.avatar
+        let header = account.header
+        imagesTask = Task {
+            await updateImages(avatar: avatar, header: header)
         }
         
         moreButton.menu = UIMenu(title: "", image: nil, identifier: nil, options: [], children: delegate?.actionsForProfile(accountID: accountID, source: .view(moreButton), fetchRelationship: false) ?? [])
@@ -290,6 +294,44 @@ class ProfileHeaderView: UIView {
         avatarContainerView.layer.cornerRadius = Preferences.shared.avatarStyle.cornerRadius(for: avatarContainerView)
         avatarImageView.layer.cornerRadius = Preferences.shared.avatarStyle.cornerRadius(for: avatarImageView)
         displayNameLabel.updateForAccountDisplayName(account: account)
+        
+        if isGrayscale != Preferences.shared.grayscaleImages {
+            isGrayscale = Preferences.shared.grayscaleImages
+            imagesTask?.cancel()
+            let avatar = account.avatar
+            let header = account.header
+            imagesTask = Task {
+                await updateImages(avatar: avatar, header: header)
+            }
+        }
+    }
+    
+    private nonisolated func updateImages(avatar: URL?, header: URL?) async {
+        await withTaskGroup(of: Void.self) { group in
+            group.addTask {
+                guard let avatar,
+                      let image = await ImageCache.avatars.get(avatar, loadOriginal: true).1,
+                      let transformedImage = await ImageGrayscalifier.convertIfNecessary(url: avatar, image: image),
+                      !Task.isCancelled else {
+                    return
+                }
+                await MainActor.run {
+                    self.avatarImageView.image = transformedImage
+                }
+            }
+            group.addTask {
+                guard let header,
+                      let image = await ImageCache.avatars.get(header, loadOriginal: true).1,
+                      let transformedImage = await ImageGrayscalifier.convertIfNecessary(url: header, image: image),
+                      !Task.isCancelled else {
+                    return
+                }
+                await MainActor.run {
+                    self.headerImageView.image = transformedImage
+                }
+            }
+            await group.waitForAll()
+        }
     }
     
     private func formatBigNumber(_ value: Int) -> (String, String) {

Tusker/Views/Profile Header/ProfileHeaderView.xib

@@ -3,7 +3,7 @@
     <device id="retina6_1" orientation="portrait" appearance="light"/>
     <dependencies>
         <deployment identifier="iOS"/>
-        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="22685"/>
+        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="22684"/>
         <capability name="Safe area layout guides" minToolsVersion="9.0"/>
         <capability name="System colors in document resources" minToolsVersion="11.0"/>
         <capability name="documents saved in the Xcode 8 format" minToolsVersion="8.0"/>
@@ -14,7 +14,7 @@
         <view contentMode="scaleToFill" translatesAutoresizingMaskIntoConstraints="NO" id="iN0-l3-epB" customClass="ProfileHeaderView" customModule="Tusker" customModuleProvider="target">
             <rect key="frame" x="0.0" y="0.0" width="414" height="896"/>
             <subviews>
-                <imageView clipsSubviews="YES" userInteractionEnabled="NO" contentMode="scaleAspectFill" horizontalHuggingPriority="251" verticalHuggingPriority="251" translatesAutoresizingMaskIntoConstraints="NO" id="dgG-dR-lSv" customClass="CachedImageView" customModule="Tusker" customModuleProvider="target">
+                <imageView clipsSubviews="YES" userInteractionEnabled="NO" contentMode="scaleAspectFill" horizontalHuggingPriority="251" verticalHuggingPriority="251" translatesAutoresizingMaskIntoConstraints="NO" id="dgG-dR-lSv">
                     <rect key="frame" x="0.0" y="48" width="414" height="150"/>
                     <constraints>
                         <constraint firstAttribute="height" constant="150" id="aCE-CA-XWm"/>
@@ -23,7 +23,7 @@
                 <view contentMode="scaleToFill" translatesAutoresizingMaskIntoConstraints="NO" id="wT9-2J-uSY">
                     <rect key="frame" x="16" y="138" width="120" height="120"/>
                     <subviews>
-                        <imageView clipsSubviews="YES" userInteractionEnabled="NO" contentMode="scaleAspectFill" horizontalHuggingPriority="251" verticalHuggingPriority="251" translatesAutoresizingMaskIntoConstraints="NO" id="TkY-oK-if4" customClass="CachedImageView" customModule="Tusker" customModuleProvider="target">
+                        <imageView clipsSubviews="YES" userInteractionEnabled="NO" contentMode="scaleAspectFill" horizontalHuggingPriority="251" verticalHuggingPriority="251" translatesAutoresizingMaskIntoConstraints="NO" id="TkY-oK-if4">
                             <rect key="frame" x="2" y="2" width="116" height="116"/>
                             <constraints>
                                 <constraint firstAttribute="height" constant="116" id="eDg-Vc-o8R"/>

Tusker/Views/Status/ConversationMainStatusCollectionViewCell.swift

@@ -640,7 +640,7 @@ extension ConversationMainStatusCollectionViewCell: UIPointerInteractionDelegate
             return defaultRegion
         } else if let button = interaction.view as? UIButton,
                   actionButtons.contains(button) {
-            var rect = button.convert(button.imageView!.bounds, from: button.imageView!)
+            var rect = button.convert(button.imageView!.bounds, to: button.imageView!)
             rect = rect.insetBy(dx: -24, dy: -24)
             return UIPointerRegion(rect: rect)
         }
@@ -654,8 +654,8 @@ extension ConversationMainStatusCollectionViewCell: UIPointerInteractionDelegate
         } else if let button = interaction.view as? UIButton,
                   actionButtons.contains(button) {
             let preview = UITargetedPreview(view: button.imageView!)
-            var rect = button.convert(button.imageView!.bounds, from: button.imageView!)
-            rect = rect.insetBy(dx: -8, dy: -8)
+            var rect = button.convert(button.imageView!.bounds, to: button.imageView!)
+            rect = rect.insetBy(dx: -24, dy: -24)
             return UIPointerStyle(effect: .highlight(preview), shape: .roundedRect(rect))
         }
         return nil