Packages/GalleryVC/Sources/GalleryVC/GalleryContentViewController.swift

@@ -18,8 +18,6 @@ public protocol GalleryContentViewController: UIViewController {
     var canAnimateFromSourceView: Bool { get }
     
     func setControlsVisible(_ visible: Bool, animated: Bool)
-    func galleryContentDidAppear()
-    func galleryContentWillDisappear()
 }
 
 public extension GalleryContentViewController {
@@ -37,10 +35,4 @@ public extension GalleryContentViewController {
     
     func setControlsVisible(_ visible: Bool, animated: Bool) {
     }
-    
-    func galleryContentDidAppear() {
-    }
-    
-    func galleryContentWillDisappear() {
-    }
 }

Packages/GalleryVC/Sources/GalleryVC/GalleryPresentationAnimationController.swift

@@ -9,11 +9,16 @@ import UIKit
 
 class GalleryPresentationAnimationController: NSObject, UIViewControllerAnimatedTransitioning {
     private let sourceView: UIView
+    private var completionHandlers: [() -> Void] = []
     
     init(sourceView: UIView) {
         self.sourceView = sourceView
     }
     
+    func addCompletionHandler(_ block: @escaping () -> Void) {
+        completionHandlers.append(block)
+    }
+    
     func transitionDuration(using transitionContext: UIViewControllerContextTransitioning?) -> TimeInterval {
         return 0.4
     }
@@ -23,6 +28,8 @@ class GalleryPresentationAnimationController: NSObject, UIViewControllerAnimated
             fatalError()
         }
         
+        to.presentationAnimationController = self
+        
         let itemViewController = to.currentItemViewController
 
         if !itemViewController.content.canAnimateFromSourceView || UIAccessibility.prefersCrossFadeTransitions {
@@ -58,13 +65,14 @@ class GalleryPresentationAnimationController: NSObject, UIViewControllerAnimated
         
         let content = itemViewController.takeContent()
         content.view.translatesAutoresizingMaskIntoConstraints = true
-        container.insertSubview(content.view, belowSubview: to.view)
-
+        
         // Use a separate dimming view from to.view, so that the gallery controls can be in front of the moving content.
         let dimmingView = UIView()
         dimmingView.backgroundColor = .black
         dimmingView.frame = container.bounds
         dimmingView.layer.opacity = 0
+        
+        container.insertSubview(content.view, belowSubview: to.view)
         container.insertSubview(dimmingView, belowSubview: content.view)
 
         to.view.backgroundColor = nil
@@ -110,7 +118,11 @@ class GalleryPresentationAnimationController: NSObject, UIViewControllerAnimated
             
             transitionContext.completeTransition(true)
             
-            to.presentationAnimationCompleted()
+            for block in self.completionHandlers {
+                block()
+            }
+            
+            to.presentationAnimationController = nil
         }
         
         animator.startAnimation()
@@ -132,7 +144,11 @@ class GalleryPresentationAnimationController: NSObject, UIViewControllerAnimated
         animator.addCompletion { _ in
             transitionContext.completeTransition(!transitionContext.transitionWasCancelled)
             
-            to.presentationAnimationCompleted()
+            for block in self.completionHandlers {
+                block()
+            }
+            
+            to.presentationAnimationController = nil
         }
         animator.startAnimation()
     }

Packages/GalleryVC/Sources/GalleryVC/GalleryViewController.swift

@@ -24,7 +24,7 @@ public class GalleryViewController: UIPageViewController {
     }
     
     private var dismissInteraction: GalleryDismissInteraction!
-    private var presentationAnimationCompletionHandlers: [() -> Void] = []
+    var presentationAnimationController: GalleryPresentationAnimationController?
     
     override public var prefersStatusBarHidden: Bool {
         true
@@ -54,7 +54,7 @@ public class GalleryViewController: UIPageViewController {
         fatalError("init(coder:) has not been implemented")
     }
     
-    public override func viewDidLoad() {
+    override public func viewDidLoad() {
         super.viewDidLoad()
         
         dismissInteraction = GalleryDismissInteraction(viewController: self)
@@ -63,30 +63,14 @@ public class GalleryViewController: UIPageViewController {
         overrideUserInterfaceStyle = .dark
         
         dataSource = self
-        delegate = self
         
         setViewControllers([makeItemVC(index: initialItemIndex)], direction: .forward, animated: false)
     }
     
-    public override func viewWillDisappear(_ animated: Bool) {
-        super.viewWillDisappear(animated)
-        
-        if isBeingDismissed {
-            currentItemViewController.content.galleryContentWillDisappear()
-        }
-    }
-    
     private func makeItemVC(index: Int) -> GalleryItemViewController {
         let content = galleryDataSource.galleryContentViewController(forItemAt: index)
         return GalleryItemViewController(delegate: self, itemIndex: index, content: content)
     }
-    
-    func presentationAnimationCompleted() {
-        for block in presentationAnimationCompletionHandlers {
-            block()
-        }
-        currentItemViewController.content.galleryContentDidAppear()
-    }
 }
 
 extension GalleryViewController: UIPageViewControllerDataSource {
@@ -111,23 +95,13 @@ extension GalleryViewController: UIPageViewControllerDataSource {
     }
 }
 
-extension GalleryViewController: UIPageViewControllerDelegate {
-    public func pageViewController(_ pageViewController: UIPageViewController, willTransitionTo pendingViewControllers: [UIViewController]) {
-        currentItemViewController.content.galleryContentWillDisappear()
-    }
-    
-    public func pageViewController(_ pageViewController: UIPageViewController, didFinishAnimating finished: Bool, previousViewControllers: [UIViewController], transitionCompleted completed: Bool) {
-        currentItemViewController.content.galleryContentDidAppear()
-    }
-}
-
 extension GalleryViewController: GalleryItemViewControllerDelegate {
     func isGalleryBeingPresented() -> Bool {
         isBeingPresented
     }
     
     func addPresentationAnimationCompletion(_ block: @escaping () -> Void) {
-        presentationAnimationCompletionHandlers.append(block)
+        presentationAnimationController?.addCompletionHandler(block)
     }
     
     func galleryItemClose(_ item: GalleryItemViewController) {

Tusker/Screens/Gallery/LoadingGalleryContentViewController.swift

@@ -104,12 +104,4 @@ class LoadingGalleryContentViewController: UIViewController, GalleryContentViewC
         wrapped?.setControlsVisible(visible, animated: animated)
     }
     
-    func galleryContentDidAppear() {
-        wrapped?.galleryContentDidAppear()
-    }
-    
-    func galleryContentWillDisappear() {
-        wrapped?.galleryContentWillDisappear()
-    }
-    
 }

Tusker/Screens/Gallery/VideoGalleryContentViewController.swift

@@ -87,6 +87,21 @@ class VideoGalleryContentViewController: UIViewController, GalleryContentViewCon
         preferredContentSize = item.presentationSize
     }
     
+    override func viewWillAppear(_ animated: Bool) {
+        super.viewWillAppear(animated)
+        
+        if isFirstAppearance {
+            isFirstAppearance = false
+            player.play()
+        }
+    }
+    
+    override func viewWillDisappear(_ animated: Bool) {
+        super.viewWillDisappear(animated)
+        
+        player.pause()
+    }
+    
     // MARK: GalleryContentViewController
     
     var container: (any GalleryVC.GalleryContentViewControllerContainer)?
@@ -111,17 +126,6 @@ class VideoGalleryContentViewController: UIViewController, GalleryContentViewCon
         hideControlsWorkItem?.cancel()
     }
     
-    func galleryContentDidAppear() {
-        if isFirstAppearance {
-            isFirstAppearance = false
-            player.play()
-        }
-    }
-    
-    func galleryContentWillDisappear() {
-        player.pause()
-    }
-    
 }
 
 private class PlayerView: UIView {

Tusker/Views/Attachments/AttachmentsContainerView.swift

@@ -57,7 +57,6 @@ class AttachmentsContainerView: UIView {
     private func commonInit() {
         self.isUserInteractionEnabled = true
         self.layer.cornerRadius = 5
-        self.layer.cornerCurve = .continuous
         
         createBlurView()
         createHideButton()
@@ -324,9 +323,6 @@ class AttachmentsContainerView: UIView {
         let blur = UIBlurEffect(style: .dark)
         let blurView = UIVisualEffectView(effect: blur)
         blurView.alpha = 0
-        blurView.layer.cornerRadius = 5
-        blurView.layer.cornerCurve = .continuous
-        blurView.layer.masksToBounds = true
         blurView.translatesAutoresizingMaskIntoConstraints = false
         fillView(blurView)
         let vibrancyView = UIVisualEffectView(effect: UIVibrancyEffect(blurEffect: blur, style: .label))