Code review from kb

ListableUI/Sources/Item/ItemContentCoordinator.swift

@@ -150,6 +150,9 @@ public final class ItemContentCoordinatorActions<Content:ItemContent>
     /// Allows you to update the displayed item via the provided closure, with an optional
     /// animation or delay.
     ///
+    /// Note that the `update` callback is invoked after the provided `delay`, and
+    /// is passed the value of the `Item` at that point in time.
+    ///
     /// ```
     /// func wasSelected() {
     ///    self.update(animation: .animated(0.15), after: 1.0) { item in
@@ -160,17 +163,19 @@ public final class ItemContentCoordinatorActions<Content:ItemContent>
     public func update(
         animation: ViewAnimation = .default,
         after delay: TimeInterval = 0,
-        update : (inout Item<Content>) -> ()
+        update : @escaping (inout Item<Content>) -> ()
     ) {
-        var new = self.currentProvider()
-
-        update(&new)
-
         if delay > 0 {
             Timer.scheduledTimer(withTimeInterval: delay, repeats: false) { _ in
+                var new = self.currentProvider()
+                update(&new)
+
                 self.updateCallback(new, animation)
             }
         } else {
+            var new = self.currentProvider()
+            update(&new)
+
             self.updateCallback(new, animation)
         }
     }

ListableUI/Sources/ViewAnimation.swift

@@ -23,8 +23,8 @@ public enum ViewAnimation {
     case animated(TimeInterval = 0.25, options : Set<AnimationOptions> = .default)
 
     /// A spring based animation is performed.
-    /// The default parameters are 0.25 seconds and `UISpringTimingParameters()` timing.
-    case spring(TimeInterval = 0.25, timing : UISpringTimingParameters = .init())
+    /// The default value is `UISpringTimingParameters()`.
+    case spring(UISpringTimingParameters = .init())
 
     /// Ands the animation with the provided bool, returning the animation if true, and `.none` if false.
     public func and(with animated : Bool) -> ViewAnimation {
@@ -58,8 +58,8 @@ public enum ViewAnimation {
                 }
             )
 
-        case .spring(let duration, let timing):
-            let animator = UIViewPropertyAnimator(duration: duration, timingParameters: timing)
+        case .spring(let timing):
+            let animator = UIViewPropertyAnimator(duration: 0, timingParameters: timing)
 
             animator.addAnimations(animations)
 

ListableUI/Tests/Item/ItemContentCoordinatorTests.swift

@@ -29,7 +29,11 @@ class ItemContentCoordinatorActionsTests : XCTestCase
                 $0.content.value = "update2"
             }
 
-            XCTAssertEqual(item.content.value, "update2")
+            actions.update {
+                $0.content.value = "update3"
+            }
+
+            XCTAssertEqual(item.content.value, "update3")
             XCTAssertEqual(callbackCount, 2)
         }
     }