From 68550976b90bccdebb47c150a4cbe4cd188534d3 Mon Sep 17 00:00:00 2001
From: Guillermo Chiacchio <7842501+gchiacchio@users.noreply.github.com>
Date: Wed, 12 Nov 2025 19:55:53 -0300
Subject: [PATCH 1/2] Allow tapGesture activation for FlowLink navigation

---
 Sources/FlowStack/FlowLink.swift | 74 ++++++++++++++++++++++++--------
 1 file changed, 55 insertions(+), 19 deletions(-)

diff --git a/Sources/FlowStack/FlowLink.swift b/Sources/FlowStack/FlowLink.swift
index 18a9fbe..e770eae 100644
--- a/Sources/FlowStack/FlowLink.swift
+++ b/Sources/FlowStack/FlowLink.swift
@@ -244,6 +244,10 @@ public struct FlowLink<Label>: View where Label: View {
         let zoomStyle: ZoomStyle
     }
 
+    public enum Activation { case overlayButton, tapGesture }
+
+    private var activation: Activation = .overlayButton
+
     var label: () -> Label
 
     private var value: (any (Equatable & Hashable))?
@@ -285,11 +289,13 @@ public struct FlowLink<Label>: View where Label: View {
     /// - Parameters:
     ///   - value: An optional value to present.
     ///   - configuration: An object that allows for customization of the flow transition appearance. For example, matching the transition corner radius with that of the FlowLink Label contents.
+    ///   - activation: The activation mode, either `.overlayButton` or `.tapGesture`.
     ///   - label: A label that describes the view that this link presents.
-    public init<P>(value: P?, configuration: Configuration = .init(), @ViewBuilder label: @escaping () -> Label) where P: Hashable, P: Equatable {
+    public init<P>(value: P?, configuration: Configuration = .init(), activation: Activation = .overlayButton, @ViewBuilder label: @escaping () -> Label) where P: Hashable, P: Equatable {
         self.label = label
         self.value = value
         self.configuration = configuration
+        self.activation = activation
     }
 
     var isContainedInPath: Bool {
@@ -363,27 +369,45 @@ public struct FlowLink<Label>: View where Label: View {
         return croppedImage
     }
 
-    private var button: some View {
-        label()
-            .onButtonGesture {
-                buttonPressed = true
-                // check for sibling elements and return early if we already have a presented element at this depth
-                guard !hasSiblingElement else { return }
-                Task {
-                    if configuration.transitionFromSnapshot {
-                        initSnapshots()
-                        self.context?.snapshotDict = snapshots
-                        self.context?.snapshot = snapshots[colorScheme]
-                    }
+    private func trigger() {
+        buttonPressed = true
+        // check for sibling elements and return early if we already have a presented element at this depth
+        guard !hasSiblingElement else { return }
+        Task {
+            if configuration.transitionFromSnapshot {
+                initSnapshots()
+                self.context?.snapshotDict = snapshots
+                self.context?.snapshot = snapshots[colorScheme]
+            }
 
-                    if let value = value {
-                        withTransaction(transaction) {
-                            path?.wrappedValue.append(value, context: context)
-                        }
-                    }
+            if let value = value {
+                withTransaction(transaction) {
+                    path?.wrappedValue.append(value, context: context)
                 }
             }
-            .id(refreshButton)
+        }
+    }
+
+    private var button: some View {
+        switch activation {
+        case .overlayButton:
+            return AnyView(
+                label()
+                    .onButtonGesture {
+                        trigger()
+                    }
+                    .id(refreshButton)
+            )
+        case .tapGesture:
+            return AnyView(
+                label()
+                    .contentShape(Rectangle())
+                    .onTapGesture {
+                        trigger()
+                    }
+                    .id(refreshButton)
+            )
+        }
     }
     public var body: some View {
         Group {
@@ -489,3 +513,15 @@ private extension View {
         modifier(IgnoreAnimationModifier(transition: transition))
     }
 }
+
+
+public extension View {
+    /// Wraps this view in a FlowLink powered by a SwiftUI tap gesture (no overlay UIButton),
+    /// preserving FlowLink's animation/context behavior while allowing inner Buttons to receive touches.
+    func flowTap<P>(
+        value: P?,
+        configuration: FlowLink<Self>.Configuration = .init()
+    ) -> some View where P: Hashable & Equatable {
+        FlowLink(value: value, configuration: configuration, activation: .tapGesture) { self }
+    }
+}

From ec09cca3fc8203254dca6368968956c2de1292d3 Mon Sep 17 00:00:00 2001
From: Guillermo Chiacchio <7842501+gchiacchio@users.noreply.github.com>
Date: Wed, 12 Nov 2025 21:06:31 -0300
Subject: [PATCH 2/2] Code cleanup

---
 Sources/FlowStack/FlowLink.swift | 40 +++++++++++---------------------
 1 file changed, 13 insertions(+), 27 deletions(-)

diff --git a/Sources/FlowStack/FlowLink.swift b/Sources/FlowStack/FlowLink.swift
index e770eae..cf41396 100644
--- a/Sources/FlowStack/FlowLink.swift
+++ b/Sources/FlowStack/FlowLink.swift
@@ -388,27 +388,25 @@ public struct FlowLink<Label>: View where Label: View {
         }
     }
 
+    @ViewBuilder
     private var button: some View {
         switch activation {
         case .overlayButton:
-            return AnyView(
-                label()
-                    .onButtonGesture {
-                        trigger()
-                    }
-                    .id(refreshButton)
-            )
+            label()
+                .onButtonGesture {
+                    trigger()
+                }
+                .id(refreshButton)
         case .tapGesture:
-            return AnyView(
-                label()
-                    .contentShape(Rectangle())
-                    .onTapGesture {
-                        trigger()
-                    }
-                    .id(refreshButton)
-            )
+            label()
+                .contentShape(Rectangle())
+                .onTapGesture {
+                    trigger()
+                }
+                .id(refreshButton)
         }
     }
+
     public var body: some View {
         Group {
             if isContainedInPath && configuration.animateFromAnchor {
@@ -513,15 +511,3 @@ private extension View {
         modifier(IgnoreAnimationModifier(transition: transition))
     }
 }
-
-
-public extension View {
-    /// Wraps this view in a FlowLink powered by a SwiftUI tap gesture (no overlay UIButton),
-    /// preserving FlowLink's animation/context behavior while allowing inner Buttons to receive touches.
-    func flowTap<P>(
-        value: P?,
-        configuration: FlowLink<Self>.Configuration = .init()
-    ) -> some View where P: Hashable & Equatable {
-        FlowLink(value: value, configuration: configuration, activation: .tapGesture) { self }
-    }
-}