Opt-in the dismissSwizzling call as needed

.github/workflows/ci.yml

@@ -18,8 +18,11 @@ jobs:
       fail-fast: false
       matrix:
         include:
+          - swift: "5.9"
+            xcode: "15.0.1"
+            runsOn: macos-13
           - swift: "5.8"
-            xcode: "14.3"
+            xcode: "14.3.1"
             runsOn: macos-13
           - swift: "5.7"
             xcode: "14.1"
@@ -52,6 +55,11 @@ jobs:
       fail-fast: false
       matrix:
         include:
+          - os: "17.0.1"
+            xcode: "15.0.1"
+            sim: "iPhone 15 Pro"
+            parallel: NO  # Stop random test job failures
+            runsOn: macos-13
           - os: "16.4"
             xcode: "14.3.1"
             sim: "iPhone 14 Pro"

Examples/Maps-SwiftUI/Maps/FloatingPanel/FloatingPanelView.swift

@@ -82,7 +82,10 @@ struct FloatingPanelView<Content: View, FloatingPanelContent: View>: UIViewContr
     /// Responsible to setup the view hierarchy and floating panel.
     final class Coordinator {
         private let parent: FloatingPanelView<Content, FloatingPanelContent>
-        private lazy var fpc = FloatingPanelController()
+        private lazy var fpc = {
+            FloatingPanelController.enableDismissToRemove()
+            return FloatingPanelController()
+        }()
 
         init(parent: FloatingPanelView<Content, FloatingPanelContent>) {
             self.parent = parent

Examples/Maps/Maps/AppDelegate.swift

@@ -1,9 +1,14 @@
 // Copyright 2018 the FloatingPanel authors. All rights reserved. MIT license.
 
 import UIKit
+import FloatingPanel
 
 @UIApplicationMain
 class AppDelegate: UIResponder, UIApplicationDelegate {
     var window: UIWindow?
-}
 
+    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey : Any]? = nil) -> Bool {
+        FloatingPanelController.enableDismissToRemove()
+        return true
+    }
+}

Examples/Maps/Maps/MainViewController.swift

@@ -124,7 +124,7 @@ extension MainViewController: UISearchBarDelegate {
         searchBar.showsCancelButton = true
         searchVC.showHeader(animated: true)
         searchVC.tableView.alpha = 1.0
-        detailVC.dismiss(animated: true, completion: nil)
+        detailFpc.removePanelFromParent(animated: true)
     }
     func deactivate(searchBar: UISearchBar) {
         searchBar.resignFirstResponder()

Examples/Samples/Sources/AppDelegate.swift

@@ -1,8 +1,13 @@
 // Copyright 2018 the FloatingPanel authors. All rights reserved. MIT license.
 
 import UIKit
+import FloatingPanel
 
 @UIApplicationMain
 class AppDelegate: UIResponder, UIApplicationDelegate {
     var window: UIWindow?
+    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey : Any]? = nil) -> Bool {
+        FloatingPanelController.enableDismissToRemove()
+        return true
+    }
 }

Examples/SamplesObjC/SamplesObjC/AppDelegate.m

@@ -1,9 +1,15 @@
 // Copyright 2018 the FloatingPanel authors. All rights reserved. MIT license.
 
 #import "AppDelegate.h"
+@import FloatingPanel;
 
 @interface AppDelegate ()
 @end
 
 @implementation AppDelegate
+- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary<UIApplicationLaunchOptionsKey,id> *)launchOptions
+{
+    [FloatingPanelController enableDismissToRemove];
+    return YES;
+}
 @end

Examples/Stocks/Stocks/MainViewController.swift

@@ -81,7 +81,7 @@ class MainViewController: UIViewController, FloatingPanelControllerDelegate {
     }
 
     private func hideStockTickerBanner() {
-        // Dimiss top bar with dissolve animation
+        // Dismiss top bar with dissolve animation
         UIView.animate(withDuration: 0.25) {
             self.topBannerView.alpha = 0.0
             self.labelStackView.alpha = 1.0

FloatingPanel.xcodeproj/project.pbxproj

@@ -468,6 +468,7 @@
 					"@executable_path/Frameworks",
 					"@loader_path/Frameworks",
 				);
+				MACOSX_DEPLOYMENT_TARGET = "$(RECOMMENDED_MACOSX_DEPLOYMENT_TARGET)";
 				PRODUCT_BUNDLE_IDENTIFIER = com.scenee.FloatingPanel;
 				PRODUCT_NAME = "$(TARGET_NAME:c99extidentifier)";
 				SKIP_INSTALL = YES;
@@ -499,6 +500,7 @@
 					"@executable_path/Frameworks",
 					"@loader_path/Frameworks",
 				);
+				MACOSX_DEPLOYMENT_TARGET = "$(RECOMMENDED_MACOSX_DEPLOYMENT_TARGET)";
 				PRODUCT_BUNDLE_IDENTIFIER = com.scenee.FloatingPanel;
 				PRODUCT_NAME = "$(TARGET_NAME:c99extidentifier)";
 				SKIP_INSTALL = YES;
@@ -521,6 +523,7 @@
 					"@executable_path/Frameworks",
 					"@loader_path/Frameworks",
 				);
+				MACOSX_DEPLOYMENT_TARGET = "$(RECOMMENDED_MACOSX_DEPLOYMENT_TARGET)";
 				PRODUCT_BUNDLE_IDENTIFIER = com.scenee.FloatingPanelTests;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SWIFT_VERSION = 5.0;
@@ -541,6 +544,7 @@
 					"@executable_path/Frameworks",
 					"@loader_path/Frameworks",
 				);
+				MACOSX_DEPLOYMENT_TARGET = "$(RECOMMENDED_MACOSX_DEPLOYMENT_TARGET)";
 				PRODUCT_BUNDLE_IDENTIFIER = com.scenee.FloatingPanelTests;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SWIFT_VERSION = 5.0;
@@ -633,6 +637,7 @@
 					"@executable_path/Frameworks",
 					"@loader_path/Frameworks",
 				);
+				MACOSX_DEPLOYMENT_TARGET = "$(RECOMMENDED_MACOSX_DEPLOYMENT_TARGET)";
 				PRODUCT_BUNDLE_IDENTIFIER = com.scenee.FloatingPanel;
 				PRODUCT_NAME = "$(TARGET_NAME:c99extidentifier)";
 				SKIP_INSTALL = YES;
@@ -657,6 +662,7 @@
 					"@executable_path/Frameworks",
 					"@loader_path/Frameworks",
 				);
+				MACOSX_DEPLOYMENT_TARGET = "$(RECOMMENDED_MACOSX_DEPLOYMENT_TARGET)";
 				PRODUCT_BUNDLE_IDENTIFIER = com.scenee.FloatingPanelTests;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SWIFT_VERSION = 5.0;

Sources/Controller.swift

@@ -288,8 +288,6 @@ open class FloatingPanelController: UIViewController {
     }
 
     private func setUp() {
-        _ = FloatingPanelController.dismissSwizzling
-
         modalPresentationStyle = .custom
         transitioningDelegate = modalTransition
 
@@ -310,7 +308,7 @@ open class FloatingPanelController: UIViewController {
         }
     }
 
-    // MARK:- Overrides
+    // MARK: - Overrides
 
     /// Creates the view that the controller manages.
     open override func loadView() {
@@ -381,7 +379,8 @@ open class FloatingPanelController: UIViewController {
         safeAreaInsetsObservation = nil
     }
 
-    // MARK:- Child view controller to consult
+    // MARK: - Child view controller to consult
+
     open override var childForStatusBarStyle: UIViewController? {
         return contentViewController
     }
@@ -398,19 +397,19 @@ open class FloatingPanelController: UIViewController {
         return contentViewController
     }
 
-    // MARK:- Privates
+    // MARK: - Privates
 
     private func shouldUpdateLayout(from previous: UITraitCollection, to new: UITraitCollection) -> Bool {
         return previous.horizontalSizeClass != new.horizontalSizeClass
-            || previous.verticalSizeClass != new.verticalSizeClass
-            || previous.preferredContentSizeCategory != new.preferredContentSizeCategory
-            || previous.layoutDirection != new.layoutDirection
+        || previous.verticalSizeClass != new.verticalSizeClass
+        || previous.preferredContentSizeCategory != new.preferredContentSizeCategory
+        || previous.layoutDirection != new.layoutDirection
     }
 
     private func update(safeAreaInsets: UIEdgeInsets) {
         guard
             preSafeAreaInsets != safeAreaInsets
-            else { return }
+        else { return }
 
         os_log(msg, log: devLog, type: .debug, "Update safeAreaInsets = \(safeAreaInsets)")
 
@@ -540,7 +539,7 @@ open class FloatingPanelController: UIViewController {
             self.view.leftAnchor.constraint(equalTo: parent.view.leftAnchor, constant: 0.0),
             self.view.rightAnchor.constraint(equalTo: parent.view.rightAnchor, constant: 0.0),
             self.view.bottomAnchor.constraint(equalTo: parent.view.bottomAnchor, constant: 0.0),
-            ])
+        ])
 
         show(animated: animated) { [weak self] in
             guard let self = self else { return }
@@ -694,6 +693,18 @@ open class FloatingPanelController: UIViewController {
         get { floatingPanel.layoutAdapter.surfaceLocation }
         set { floatingPanel.layoutAdapter.surfaceLocation = newValue }
     }
+
+
+    /// Calling this will allow to invoke `removePanelFromParent(animated:completion:)` as needed by
+    /// calling UIViewController's `dismiss` method
+    ///
+    /// Previously, until v2.8, this was the default behavior. However, from v2.9 onwards, due to
+    /// identified issues when used in conjunction with other libraries, it has been made an opt-in 
+    /// feature.
+    @objc
+    public static func enableDismissToRemove() {
+        _ = FloatingPanelController.dismissSwizzling
+    }
 }
 
 extension FloatingPanelController {
@@ -732,6 +743,7 @@ private var originalDismissImp: IMP?
 private typealias DismissFunction = @convention(c) (AnyObject, Selector, Bool, (() -> Void)?) -> Void
 extension FloatingPanelController {
     private static let dismissSwizzling: Void = {
+        guard originalDismissImp == nil else { return }
         let aClass: AnyClass! = UIViewController.self //object_getClass(vc)
         if let originalMethod = class_getInstanceMethod(aClass, #selector(dismiss(animated:completion:))),
            let swizzledImp = class_getMethodImplementation(aClass, #selector(__swizzled_dismiss(animated:completion:))) {