Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes breaking app on back btn clearHistory: true on android #1637

Merged
merged 1 commit into from Feb 25, 2016

Conversation

atanasovg
Copy link
Contributor

This pull fixes #1569. The problem was that the clearHistory option resets the _currentEntry field and the

let isFirstNavigation = types.isNullOrUndefined(this._currentEntry);

expression returned true when it is actually false and this corrupts the native Android backstack.

@atanasovg
Copy link
Contributor Author

ping @hshristov @enchev

@enchev
Copy link
Contributor

enchev commented Feb 24, 2016

👍

@ns-bot ns-bot added cla: yes and removed cla: no labels Feb 24, 2016
@vchimev
Copy link
Contributor

vchimev commented Feb 24, 2016

run ci

@ErjanGavalji ErjanGavalji changed the title Fixes #1569 Fixes breaking app on back btn clearHistory: true on android Feb 25, 2016
@hshristov
Copy link
Contributor

@atanasovg I'm working on this issue and it seems there are few bugs - one crash, one memory leak and one issue where page stays empty on navigating back. So I think it will be better to fix them all at once because they seems related. Once I'm ready I will push another PR. So we can close this for the moment, ok?

@ErjanGavalji
Copy link
Contributor

👍

hshristov pushed a commit that referenced this pull request Feb 25, 2016
…learhistory

Fixes breaking app on back btn clearHistory: true on android
@hshristov hshristov merged commit b3721cc into master Feb 25, 2016
@hshristov hshristov deleted the atanasovg/fix-navigation-clearhistory branch February 25, 2016 15:22
@lock
Copy link

lock bot commented Aug 28, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Aug 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Regression: Back button breaks app on clearHistory: true (android)
6 participants