Skip to content

Fix/same bounds multi flow - #144

Merged
eds2002 merged 2 commits into
mainfrom
fix/same-bounds-multi-flow
Aug 3, 2026
Merged

Fix/same bounds multi flow#144
eds2002 merged 2 commits into
mainfrom
fix/same-bounds-multi-flow

Conversation

@eds2002

@eds2002 eds2002 commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Motivation

In multi-screen navigation flows, a completed bounds reveal transition could reactivate when pushing another route that reused the same boundary ID.

The outgoing interpolator was evaluated with a filtered view of the transition, but its bounds() and depth-zero transition() helpers still read the latest navigation frame. This mismatch allowed a settled reveal target to pair with an element on the newly pushed screen, causing it to animate out unexpectedly.

This change ensures the interpolator and its helpers use the same transition frame, preserving completed reveal states while allowing subsequent transitions to handle matching boundary IDs independently.

eds2002 added 2 commits August 3, 2026 09:42
Ensure `bounds()` and depth-zero `transition()` use the same selected
frame as the interpolator’s direct properties.

This prevents settled reveal transitions from reactivating when a later
screen reuses the same boundary ID.
@netlify

netlify Bot commented Aug 3, 2026

Copy link
Copy Markdown

Deploy Preview for rnst-docs canceled.

Name Link
🔨 Latest commit 7ce56b3
🔍 Latest deploy log https://app.netlify.com/projects/rnst-docs/deploys/6a70a08785829c00085b6008

@eds2002
eds2002 merged commit 5d62772 into main Aug 3, 2026
6 checks passed
@eds2002
eds2002 deleted the fix/same-bounds-multi-flow branch August 3, 2026 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant