debounce

Author: naftaly

Sources/EmbraceCore/SwiftUI/EmbraceTraceSurfaceView.swift

@@ -30,7 +30,7 @@ public struct EmbraceTraceSurfaceView<Content: View>: View {
 
     @StateObject
     internal var state = SurfaceState()
-
+    
     public init(
         _ name: String,
         attributes: [String: String]? = nil,
@@ -40,7 +40,7 @@ public struct EmbraceTraceSurfaceView<Content: View>: View {
         self.attributes = attributes
         self.content = content
     }
-    
+
     public var body: some View {
         content()
             .environment(\.embraceSurfaceParent, state.id)

Sources/EmbraceCore/SwiftUI/Internal/EmbraceSurfaceInternal.swift

@@ -9,6 +9,19 @@ import SwiftUI
 extension EmbraceTraceSurfaceView {
 
     internal func frameChanged() {
+        // not sure i want to debounce here.
+        // debounce causes the updates to wait
+        // until almost no movement to update
+        // the top surface which can be a bit late.
+        /*
+        state.debounceFrameUpdate {
+            performFrameUpdatesAfterDebounce()
+        }
+        */
+        performFrameUpdatesAfterDebounce()
+    }
+    
+    private func performFrameUpdatesAfterDebounce() {
 
         let newPercentage: Double
         defer {
@@ -57,6 +70,22 @@ extension EmbraceTraceSurfaceView {
         var frame: CGRect = .zero
         var window: UIWindow? = nil
 
+        private var frameUpdateWorkItem: DispatchWorkItem? = nil
+        
+        deinit {
+            frameUpdateWorkItem?.cancel()
+        }
+        
+        func debounceFrameUpdate(_ action: @escaping () -> Void) {
+            dispatchPrecondition(condition: .onQueue(.main))
+            frameUpdateWorkItem?.cancel()
+            frameUpdateWorkItem = DispatchWorkItem(block: action)
+            guard let workItem = frameUpdateWorkItem else {
+                return
+            }
+            DispatchQueue.main.asyncAfter(deadline: .now() + 0.1, execute: workItem)
+        }
+        
         var visibleBasedOnAppearance: Bool = false {
             didSet { updateVisibility() }
         }