@ionic/react-router 9.0.5 → 9.0.6-dev.11790280751.1fc334cc

package/dist/index.js

@@ -499,6 +499,41 @@ const compareRouteSpecificity = (a, b) => {
     }
     return 0;
 };
+/**
+ * True when a route matches more pathnames than its own path, so a splat, an index route,
+ * or a route with an empty or absent path. A lookup can return one of these for a pathname
+ * a more specific sibling owns, so callers must confirm ownership against React Router's
+ * ranking before reusing the view item.
+ *
+ * This is deliberately wider than the catch-all checks in ReactRouterViewStack, which each
+ * gate on a narrower shape for a different reason. Don't unify them with this helper.
+ */
+const isOverMatchingRoute = (route) => {
+    const { path, index } = route;
+    return !path || path.includes('*') || !!index;
+};
+/** True when the navigation pushed a new page forward on top of the current one. */
+const isForwardPush = (routeInfo) => routeInfo.routeAction === 'push' && routeInfo.routeDirection === 'forward';
+const swipeRevealed = new WeakSet();
+/**
+ * Marks the page a swipe-back gesture has revealed. For the length of the drag that page
+ * is on screen while a more specific sibling still matches the current pathname, and the
+ * deactivation scan in `renderViewItem` would otherwise re-hide it on the next render and
+ * leave the user dragging a blank page.
+ */
+const markSwipeRevealed = (viewItem) => {
+    if (viewItem) {
+        swipeRevealed.add(viewItem);
+    }
+};
+/** Drops the mark once the gesture ends, so the view is hidden normally again. */
+const clearSwipeRevealed = (viewItem) => {
+    if (viewItem) {
+        swipeRevealed.delete(viewItem);
+    }
+};
+/** True while a swipe-back gesture is showing this view. */
+const isSwipeRevealed = (viewItem) => swipeRevealed.has(viewItem);
 /**
  * Sorts view items by route specificity (most specific first).
  *
@@ -860,7 +895,7 @@ class ReactRouterViewStack extends ViewStacks {
         }
         // Deactivate wildcard (catch-all) and empty-path (default) routes when a more-specific route matches.
         // This prevents "Not found" or fallback pages from showing alongside valid routes.
-        if (routePath === '*' || routePath === '') {
+        if (routePath === '*' || routePath === '/*' || routePath === '') {
             // Check if any other view in this outlet has a match for the current route
             const outletViews = this.getViewItemsForOutlet(viewItem.outletId);
             // When parent path context is available, compute the relative pathname once
@@ -913,8 +948,15 @@ class ReactRouterViewStack extends ViewStacks {
                     }
                 }
             }
-            if (hasSpecificMatch) {
-                viewItem.mount = false;
+            if (hasSpecificMatch && !isSwipeRevealed(viewItem)) {
+                // A splat can be the outlet's container page rather than a "not found" fallback.
+                // Pushed over it is the page underneath, so unmounting it destroys its state and
+                // leaves nothing for back to reveal. Hiding it below covers that. A view with no
+                // ion-page was never a page in the stack and has nothing to hide, so it still
+                // unmounts.
+                if (!isForwardPush(routeInfo) || !viewItem.ionPageElement) {
+                    viewItem.mount = false;
+                }
                 if (viewItem.ionPageElement) {
                     viewItem.ionPageElement.classList.add('ion-page-hidden');
                     viewItem.ionPageElement.setAttribute('aria-hidden', 'true');
@@ -984,8 +1026,11 @@ class ReactRouterViewStack extends ViewStacks {
                 parentPath = result.parentPath;
                 // Persist the mount path for subsequent calls, mirroring StackManager.outletMountPath.
                 // Unlike outletParentPaths (cleared when parentPath is undefined), the mount path is
-                // intentionally sticky — it anchors the outlet's scope and is only removed in clear().
-                if (result.outletMountPath && !this.outletMountPaths.has(outletId)) {
+                // intentionally sticky, it anchors the outlet's scope and is only removed in clear().
+                // A root outlet is skipped because it is mounted under nothing, so an inferred path
+                // would scope it to whatever route was active. parentPathnameBase is undefined
+                // exactly when the outlet has no parent matches.
+                if (parentPathnameBase && result.outletMountPath && !this.outletMountPaths.has(outletId)) {
                     this.outletMountPaths.set(outletId, result.outletMountPath);
                 }
             }
@@ -1435,6 +1480,8 @@ class StackManager extends React.PureComponent {
     routerOutletElement;
     prevProps;
     skipTransition;
+    /** The view item the in-flight swipe gesture revealed, so the same one gets cleared. */
+    swipeRevealedViewItem;
     stackContextValue = {
         registerIonPage: this.registerIonPage.bind(this),
         isInOutlet: () => true,
@@ -1526,7 +1573,9 @@ class StackManager extends React.PureComponent {
                     hasIndexRoute,
                     hasWildcardRoute,
                 });
-                if (result.outletMountPath && !this.outletMountPath) {
+                // A root outlet is mounted under nothing, so caching the inferred path would scope
+                // it to whatever route was active and make every other pathname look out of scope.
+                if (!this.isRootOutlet && result.outletMountPath && !this.outletMountPath) {
                     this.outletMountPath = result.outletMountPath;
                 }
                 return result.parentPath;
@@ -1581,12 +1630,16 @@ class StackManager extends React.PureComponent {
             return true;
         }
         // For non-replace actions, only unmount for back navigation
-        const isForwardPush = routeInfo.routeAction === 'push' && routeInfo.routeDirection === 'forward';
-        if (!isForwardPush && routeInfo.routeDirection !== 'none' && enteringViewItem !== leavingViewItem) {
+        if (!isForwardPush(routeInfo) && routeInfo.routeDirection !== 'none' && enteringViewItem !== leavingViewItem) {
             return true;
         }
         return false;
     }
+    /** Clears the reveal flag from the view item the gesture marked. */
+    clearSwipeRevealedView() {
+        clearSwipeRevealed(this.swipeRevealedViewItem);
+        this.swipeRevealedViewItem = undefined;
+    }
     /**
      * Handles out-of-scope outlet. Returns true if transition should be aborted.
      */
@@ -1756,7 +1809,9 @@ class StackManager extends React.PureComponent {
             const containerBase = routePath.replace(/\/\*$/, '');
             const currentInContainer = routeInfo.pathname.startsWith(containerBase + '/') || routeInfo.pathname === containerBase;
             const previousInContainer = routeInfo.lastPathname.startsWith(containerBase + '/') || routeInfo.lastPathname === containerBase;
-            if (currentInContainer && previousInContainer) {
+            // A root-level "/*" leaves an empty base, so both checks above are true for every
+            // pathname and the shortcut would otherwise skip every navigation in this outlet.
+            if (containerBase !== '' && currentInContainer && previousInContainer) {
                 const updatedMatch = matchComponent(enteringViewItem.reactElement, routeInfo.pathname, true, this.outletMountPath);
                 if (updatedMatch) {
                     enteringViewItem.routeData.match = updatedMatch;
@@ -2218,6 +2273,7 @@ class StackManager extends React.PureComponent {
         }
         this.waitingForIonPage = false;
         this.preservedViewItems.clear();
+        this.clearSwipeRevealedView();
         // Hide all views in this outlet before clearing.
         // This is critical for nested outlets - when the parent component unmounts,
         // the nested outlet's componentDidUpdate won't be called, so we must hide
@@ -2249,13 +2305,32 @@ class StackManager extends React.PureComponent {
             this.pendingPageTransition = true;
             return;
         }
+        // A completed swipe keeps its reveal flag past goBack() so the deactivation scan does
+        // not re-hide the page between the gesture ending and the URL settling. Clear it at
+        // the start of the transition for the new pathname.
+        this.clearSwipeRevealedView();
         // Find entering and leaving view items
         const viewItems = this.findViewItems(routeInfo);
         let enteringViewItem = viewItems.enteringViewItem;
         let leavingViewItem = viewItems.leavingViewItem;
-        let shouldUnmountLeavingViewItem = this.shouldUnmountLeavingView(routeInfo, enteringViewItem, leavingViewItem);
         // Get parent path for nested outlets
         const parentPath = this.getParentPath();
+        // Find the matching route element. This is React Router's own ranking of the outlet's
+        // routes, so it decides which route owns the pathname.
+        const enteringRoute = findRouteByRouteInfo(this.ionRouterOutlet?.props.children, routeInfo, parentPath);
+        // The lookup above only matches view items that already exist, so a sibling route with
+        // no view item yet never wins there and an over-matching route (splat, index, empty
+        // path) comes back for pathnames it does not own. Drop it rather than reusing it,
+        // because overwriting its reactElement below would swap its page for the winning
+        // route's and unmount it. This has to resolve before shouldUnmountLeavingView and
+        // handleRootNavigation, which both branch on the entering view item.
+        if (enteringViewItem &&
+            enteringRoute &&
+            isOverMatchingRoute(enteringViewItem.routeData?.childProps ?? {}) &&
+            enteringRoute.props.path !== enteringViewItem.routeData?.childProps?.path) {
+            enteringViewItem = undefined;
+        }
+        let shouldUnmountLeavingViewItem = this.shouldUnmountLeavingView(routeInfo, enteringViewItem, leavingViewItem);
         // Handle out-of-scope outlet (route outside mount path)
         if (this.handleOutOfScopeOutlet(routeInfo)) {
             return;
@@ -2275,8 +2350,6 @@ class StackManager extends React.PureComponent {
         if (this.handleOutOfContextNestedOutlet(parentPath, leavingViewItem)) {
             return;
         }
-        // Find the matching route element
-        const enteringRoute = findRouteByRouteInfo(this.ionRouterOutlet?.props.children, routeInfo, parentPath);
         // Handle nested outlet with no matching route
         if (this.handleNoMatchingRoute(enteringRoute, enteringViewItem, leavingViewItem)) {
             return;
@@ -2497,6 +2570,9 @@ class StackManager extends React.PureComponent {
             return canStartSwipe;
         };
         const onStart = async () => {
+            // Core only calls onEnd when a progress animation exists, so with animations off a
+            // previous gesture can leave its mark behind. Drop any stale one before marking.
+            this.clearSwipeRevealedView();
             const { routeInfo } = this.props;
             const swipeBackRouteInfo = this.getSwipeBackRouteInfo();
             const enteringViewItem = this.findEnteringViewForSwipe(swipeBackRouteInfo);
@@ -2523,6 +2599,11 @@ class StackManager extends React.PureComponent {
             revealIonPageForSwipeBack(enteringViewItem?.ionPageElement);
             // When the gesture starts, kick off a transition controlled via swipe gesture
             if (enteringViewItem && leavingViewItem) {
+                // Without a leaving view there is no progress animation, and core only calls
+                // swipeHandler.onEnd when one exists, so nothing would clear the mark until the
+                // next transition.
+                markSwipeRevealed(enteringViewItem);
+                this.swipeRevealedViewItem = enteringViewItem;
                 await this.transitionPage(routeInfo, enteringViewItem, leavingViewItem, 'back', true);
             }
             debug('SwipeBackOnStart:exit', () => ({
@@ -2547,6 +2628,7 @@ class StackManager extends React.PureComponent {
                 const swipeBackRouteInfo = this.getSwipeBackRouteInfo();
                 const enteringViewItem = this.findEnteringViewForSwipe(swipeBackRouteInfo);
                 const leavingViewItem = this.context.findViewItemByRouteInfo(routeInfo, this.id, false);
+                this.clearSwipeRevealedView();
                 // Don't hide if entering and leaving are the same (parameterized route edge case)
                 if (enteringViewItem !== leavingViewItem && enteringViewItem?.ionPageElement !== undefined) {
                     hideIonPageElement(enteringViewItem.ionPageElement);