@bluebillywig/react-native-bb-player 8.45.21 → 8.45.22

package/ios/BBShortsView.swift

@@ -139,12 +139,23 @@ class BBShortsView: UIView, BBNativeShortsViewDelegate {
 
     // Wrap in a UINavigationController so the native SDK can present
     // shelf-mode modals (openShortsPlayerAsModal uses navigationController?.present).
-    // Hide the nav bar since we don't need visible navigation chrome.
-    let nav = UINavigationController(rootViewController: UIViewController())
+    // The native SDK adds BBNativeShortsViewController as a child of the VC we pass.
+    // That child then uses its navigationController to present modals.
+    let containerVC = UIViewController()
+    containerVC.view.backgroundColor = .clear
+    let nav = UINavigationController(rootViewController: containerVC)
     nav.isNavigationBarHidden = true
     nav.view.frame = bounds
     nav.view.backgroundColor = .clear
     viewController.addChild(nav)
+    addSubview(nav.view)
+    nav.view.translatesAutoresizingMaskIntoConstraints = false
+    NSLayoutConstraint.activate([
+      nav.view.topAnchor.constraint(equalTo: topAnchor),
+      nav.view.leadingAnchor.constraint(equalTo: leadingAnchor),
+      nav.view.trailingAnchor.constraint(equalTo: trailingAnchor),
+      nav.view.bottomAnchor.constraint(equalTo: bottomAnchor),
+    ])
     nav.didMove(toParent: viewController)
     self.navController = nav
 
@@ -154,10 +165,10 @@ class BBShortsView: UIView, BBNativeShortsViewDelegate {
       optionsDict = dict
     }
 
-    // Create the shorts view using the factory method
-    // Pass the nav controller so the native SDK's shelf tap can present modals
+    // Create the shorts view using the container VC inside the nav controller
+    // so the native SDK's shelf tap can present modals via navigationController
     shortsView = BBNativeShorts.createShortsView(
-      uiViewController: nav,
+      uiViewController: containerVC,
       frame: bounds,
       jsonUrl: jsonUrl,
       options: optionsDict
@@ -171,14 +182,15 @@ class BBShortsView: UIView, BBNativeShortsViewDelegate {
 
     shorts.delegate = self
 
-    // Add to view hierarchy with autolayout
-    addSubview(shorts)
+    // Add shorts view inside the container VC's view so it sits within the
+    // navigation controller hierarchy (required for shelf modal presentation)
+    containerVC.view.addSubview(shorts)
     shorts.translatesAutoresizingMaskIntoConstraints = false
     NSLayoutConstraint.activate([
-      shorts.topAnchor.constraint(equalTo: topAnchor),
-      shorts.leadingAnchor.constraint(equalTo: leadingAnchor),
-      shorts.trailingAnchor.constraint(equalTo: trailingAnchor),
-      shorts.bottomAnchor.constraint(equalTo: bottomAnchor)
+      shorts.topAnchor.constraint(equalTo: containerVC.view.topAnchor),
+      shorts.leadingAnchor.constraint(equalTo: containerVC.view.leadingAnchor),
+      shorts.trailingAnchor.constraint(equalTo: containerVC.view.trailingAnchor),
+      shorts.bottomAnchor.constraint(equalTo: containerVC.view.bottomAnchor)
     ])
 
     log("BBShortsView.setupShorts() completed")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bluebillywig/react-native-bb-player",
-  "version": "8.45.21",
+  "version": "8.45.22",
   "description": "Blue Billywig Native Video Player for React Native - iOS AVPlayer and Android ExoPlayer integration",
   "main": "lib/commonjs/index.js",
   "module": "lib/module/index.js",