make gesture simultaneous and do not select an item if the gesture ends outside of a view

Author: mikhailChelbaev

Sources/ComponentsKit/RadioGroup/SwiftUI/SURadioGroup.swift

@@ -10,6 +10,7 @@ public struct SURadioGroup<ID: Hashable>: View {
   /// A Binding value to control the selected identifier.
   @Binding public var selectedId: ID?
 
+  @State private var viewSizes: [ID: CGSize] = [:]
   @Environment(\.colorScheme) private var colorScheme
   @State private var tappingId: ID?
 
@@ -57,14 +58,29 @@ public struct SURadioGroup<ID: Hashable>: View {
               self.model.textColor(for: item).color(for: self.colorScheme)
             )
         }
-        .gesture(
+        .background(
+          GeometryReader { proxy in
+            Color.clear
+              .onAppear {
+                self.viewSizes[item.id] = proxy.size
+              }
+              .onChange(of: proxy.size) { value in
+                self.viewSizes[item.id] = value
+              }
+          }
+        )
+        .simultaneousGesture(
           DragGesture(minimumDistance: 0)
             .onChanged { _ in
               self.tappingId = item.id
             }
-            .onEnded { _ in
+            .onEnded { gesture in
               self.tappingId = nil
-              self.selectedId = item.id
+
+              if let size = self.viewSizes[item.id],
+                 CGRect(origin: .zero, size: size).contains(gesture.location) {
+                self.selectedId = item.id
+              }
             }
         )
         .disabled(!item.isEnabled || !self.model.isEnabled)

Sources/ComponentsKit/RadioGroup/UIKit/UKRadioGroup.swift

@@ -2,7 +2,7 @@ import AutoLayout
 import UIKit
 
 /// A UIKit component that displays a group of radio buttons, allowing users to select one option from multiple choices.
-open class UKRadioGroup<ID: Hashable>: UIView, UKComponent {
+open class UKRadioGroup<ID: Hashable>: UIView, UKComponent, UIGestureRecognizerDelegate {
   // MARK: Properties
 
   /// A closure that is triggered when a selected segment changes.
@@ -81,6 +81,7 @@ open class UKRadioGroup<ID: Hashable>: UIView, UKComponent {
         action: #selector(self.handleContainerLongPress(_:))
       )
       longPressGesture.minimumPressDuration = 0
+      longPressGesture.delegate = self
       radioGroupItemView.addGestureRecognizer(longPressGesture)
 
       self.items[item.id] = radioGroupItemView
@@ -154,14 +155,26 @@ open class UKRadioGroup<ID: Hashable>: UIView, UKComponent {
     case .ended:
       self.tappingId = nil
       self.animateRadioView(for: tappedId, scale: 1.0)
-      self.selectedId = tappedId
+
+      if tappedView.bounds.contains(sender.location(in: tappedView)) {
+        self.selectedId = tappedId
+      }
     case .cancelled, .failed:
       self.tappingId = nil
       self.animateRadioView(for: tappedId, scale: 1.0)
     default:
       break
     }
   }
+
+  // MARK: UKRadioGroup + UIGestureRecognizerDelegate
+
+  public func gestureRecognizer(
+    _ gestureRecognizer: UIGestureRecognizer,
+    shouldRecognizeSimultaneouslyWith otherGestureRecognizer: UIGestureRecognizer
+  ) -> Bool {
+    return true
+  }
 }
 
 // MARK: - Style Helpers