FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(tab-view): remove onBackPressed override by ADjenkov · Pull Request #6755 · NativeScript/NativeScript · GitHub

fix(tab-view): remove onBackPressed override - #6755

Merged
ADjenkov merged 1 commit into
masterfrom
djenkov/tab-view-frame
Jan 4, 2019
Merged

fix(tab-view): remove onBackPressed override#6755
ADjenkov merged 1 commit into
masterfrom
djenkov/tab-view-frame

Conversation

ADjenkov commented Jan 4, 2019

Copy link
Copy Markdown
Contributor

Android back button will exit the app, instead going back to previous list page, when using lazy loaded nested named inside TabViewItems.

This happens only if using as a root component (template-tab-navigation-ng template).

Remove onBackPressed override, since the topmost() frame is now properly assigned when switching between tab view items.

Fix NativeScript/nativescript-angular#1680

ADjenkov added the bug label Jan 4, 2019
ADjenkov self-assigned this Jan 4, 2019
ghost added the in progress label Jan 4, 2019

ADjenkov commented Jan 4, 2019

Copy link
Copy Markdown
Contributor Author

test ngapps

dtopuzov commented Jan 4, 2019

Copy link
Copy Markdown
Contributor

test --ignore templates api23 ios12

ADjenkov merged commit 984f162 into master Jan 4, 2019
ghost removed bug in progress labels Jan 4, 2019
ADjenkov deleted the djenkov/tab-view-frame branch January 4, 2019 14:57
SvetoslavTsenov added a commit that referenced this pull request Jan 15, 2019
…ase-5.1.2

fix(tab-view): remove onBackPressed override (#6755)

lock Bot commented Jan 4, 2020

Copy link
Copy Markdown

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 Bot locked and limited conversation to collaborators Jan 4, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Navigate back with hardware button put the app in background

3 participants


Back | FazBrowse Home | New Git URL