Skip to content

Commit 48b3243

Browse files
committed
fix(navigation): mark as not transitioning on success in addition to '_transitionFinish', provide no
mark as not transitioning on success in addition to '_transitionFinish', provide note as to why we want it in both places
1 parent 486bff0 commit 48b3243

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/navigation/nav-controller-base.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -223,6 +223,8 @@ export class NavControllerBase extends Ion implements NavController {
223223
this._init = true;
224224
this._trnsId = null;
225225

226+
// ensure we're not transitioning here
227+
this.setTransitioning(false);
226228
this._swipeBackCheck();
227229
// let's see if there's another to kick off
228230
this._nextTrns();
@@ -749,6 +751,8 @@ export class NavControllerBase extends Ion implements NavController {
749751

750752
// it's safe to enable the app again
751753
this._app.setEnabled(true);
754+
// mark ourselves as not transitioning - `deepLinker navchange` requires this
755+
// TODO - probably could be resolved in a better way
752756
this.setTransitioning(false);
753757

754758
if (!this.hasChild() && opts.updateUrl !== false) {

0 commit comments

Comments
 (0)