Disable Animation in CropView (#232)

FluidGroup · Mar 29, 2024 · ca2935b · ca2935b
1 parent 1dcf0b8
commit ca2935b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/Sources/BrightroomUI/Shared/Components/Crop/CropView.swift b/Sources/BrightroomUI/Shared/Components/Crop/CropView.swift
@@ -97,6 +97,8 @@ public final class CropView: UIView, UIScrollViewDelegate {
     }
   }
 
+  public var areAnimationsEnabled: Bool = true
+
   public var isImageViewHidden: Bool {
     get {
       imagePlatterView.imageView.isHidden
@@ -332,7 +334,7 @@ public final class CropView: UIView, UIScrollViewDelegate {
       editingStack.start()
 
       binding: do {
-        store.sinkState(queue: .mainIsolated()) { [weak self] state in
+        store.sinkState(queue: .mainIsolated()) { [weak self, areAnimationsEnabled] state in
 
           guard let self = self else { return }
 
@@ -379,7 +381,7 @@ public final class CropView: UIView, UIScrollViewDelegate {
             self.updateScrollContainerView(
               by: crop,
               preferredAspectRatio: state.preferredAspectRatio,
-              animated: state.previous?.proposedCrop != nil /* whether first time load */,
+              animated: areAnimationsEnabled && state.previous?.proposedCrop != nil /* whether first time load */,
               animatesRotation: state.hasChanges(\.proposedCrop?.rotation)
             )
 

diff --git a/Sources/BrightroomUI/Shared/Components/Drawing/BlurryMaskingView.swift b/Sources/BrightroomUI/Shared/Components/Drawing/BlurryMaskingView.swift
@@ -83,7 +83,7 @@ public final class BlurryMaskingView: PixelEditorCodeBasedView, UIScrollViewDele
     }
   }
 
-  let backingView: CropView
+  private let backingView: CropView
 
   private let containerView = ContainerView()
 
@@ -115,6 +115,7 @@ public final class BlurryMaskingView: PixelEditorCodeBasedView, UIScrollViewDele
       editingStack: editingStack,
       contentInset: .zero
     )
+    self.backingView.areAnimationsEnabled = false
     self.backingView.accessibilityIdentifier = "BlurryMasking"
 
     store = .init(