Align .speed modifier for Cocoa animations with SwiftUI (#278)

* Fix .speed modifier for CocoaAnimations

- Align with SwiftUI animations
- Add issue reporting

* Refactor AppKitAnimation extensions and structure

Refactor AppKitAnimation by removing unnecessary closing brace and adjusting extensions.

* Clarify warning for zero animation speed

Updated warning message for animation speed adjustment.

* Fix warning message for zero animation speed

* Set default speed to 1 in AppKitAnimation.swift

---------

Co-authored-by: Stephen Celis <[email protected]>
Co-authored-by: Stephen Celis <[email protected]>

Author: maximkrouk

Sources/AppKitNavigation/AppKitAnimation.swift

@@ -1,6 +1,7 @@
 #if canImport(AppKit) && !targetEnvironment(macCatalyst)
   import AppKit
   import SwiftNavigation
+  import IssueReporting
 
   #if canImport(SwiftUI)
     import SwiftUI
@@ -43,7 +44,7 @@
         var result: Swift.Result<Result, Error>?
         NSAnimationContext.runAnimationGroup { context in
           context.allowsImplicitAnimation = true
-          context.duration = animation.duration
+          context.duration = animation.duration / animation.speed
           context.timingFunction = animation.timingFunction
           result = Swift.Result(catching: body)
         } completionHandler: {
@@ -74,13 +75,42 @@
 
       fileprivate struct AppKit: Hashable, @unchecked Sendable {
         fileprivate var duration: TimeInterval
+        fileprivate var speed: TimeInterval = 1
         fileprivate var timingFunction: CAMediaTimingFunction?
 
         func hash(into hasher: inout Hasher) {
           hasher.combine(duration)
         }
       }
     }
+    
+    /// Changes the duration of an animation by adjusting its speed.
+    ///
+    /// - Parameter speed: The speed at which SwiftUI performs the animation.
+    /// - Returns: An animation with the adjusted speed.
+    public func speed(
+      _ speed: Double
+    ) -> Self {
+      switch framework {
+      case let .swiftUI(animation):
+        return AppKitAnimation(
+          framework: .swiftUI(animation.speed(speed))
+        )
+      case var .appKit(animation):
+        if speed != 0 {
+          animation.speed = speed
+        } else {
+          reportIssue(
+            """
+            Setting animation speed to zero is not supported for AppKit animations. \
+            Replace with '.ulpOfOne' to avoid division by zero.
+            """
+          )
+          animation.speed = .ulpOfOne
+        }
+        return AppKitAnimation(framework: .appKit(animation))
+      }
+    }
   }
 
   extension AppKitAnimation {

Sources/UIKitNavigation/UIKitAnimation.swift

@@ -1,5 +1,6 @@
 #if canImport(UIKit) && !os(watchOS)
   import UIKit
+  import IssueReporting
 
   #if canImport(SwiftUI)
     import SwiftUI
@@ -71,8 +72,8 @@
           var result: Swift.Result<Result, Error>?
           withoutActuallyEscaping(animations) { animations in
             UIView.animate(
-              withDuration: animation.duration * animation.speed,
-              delay: animation.delay * animation.speed,
+              withDuration: animation.duration / animation.speed,
+              delay: animation.delay / animation.speed,
               options: animation.options,
               animations: { result = Swift.Result(catching: animations) },
               completion: completion
@@ -84,8 +85,8 @@
           var result: Swift.Result<Result, Error>?
           withoutActuallyEscaping(animations) { animations in
             UIView.animate(
-              withDuration: animation.duration * animation.speed,
-              delay: animation.delay * animation.speed,
+              withDuration: animation.duration / animation.speed,
+              delay: animation.delay / animation.speed,
               usingSpringWithDamping: dampingRatio,
               initialSpringVelocity: velocity,
               options: animation.options,
@@ -99,10 +100,10 @@
           if #available(iOS 17, macOS 14, tvOS 17, watchOS 10, *) {
             var result: Swift.Result<Result, Error>?
             UIView.animate(
-              springDuration: animation.duration * animation.speed,
+              springDuration: animation.duration / animation.speed,
               bounce: bounce,
               initialSpringVelocity: initialSpringVelocity,
-              delay: animation.delay * animation.speed,
+              delay: animation.delay / animation.speed,
               options: animation.options,
               animations: { result = Swift.Result(catching: animations) },
               completion: completion
@@ -220,7 +221,18 @@
           framework: .swiftUI(animation.speed(speed))
         )
       case var .uiKit(animation):
-        animation.speed = speed
+        if speed != 0 {
+          animation.speed = speed
+        } else {
+          reportIssue(
+            """
+            Setting animation speed to zero is not supported for UIKit animations.
+            
+            Replace with '.ulpOfOne' to avoid division by zero.
+            """
+          )
+          animation.speed = .ulpOfOne
+        }
         return UIKitAnimation(framework: .uiKit(animation))
       }
     }