Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Merged
ADjenkov merged 1 commit into
masterNativeScript/NativeScript:masterfrom
djenkov/tab-view-frameNativeScript/NativeScript:djenkov/tab-view-frameCopy head branch name to clipboard
Jan 4, 2019
Merged

fix(tab-view): remove onBackPressed override#6755
ADjenkov merged 1 commit into
masterNativeScript/NativeScript:masterfrom
djenkov/tab-view-frameNativeScript/NativeScript:djenkov/tab-view-frameCopy head branch name to clipboard

Conversation

@ADjenkov
Copy link
Copy Markdown
Contributor

@ADjenkov ADjenkov commented Jan 4, 2019

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 ADjenkov added the bug label Jan 4, 2019
@ADjenkov ADjenkov self-assigned this Jan 4, 2019
@ghost ghost added the in progress label Jan 4, 2019
@ADjenkov
Copy link
Copy Markdown
Contributor Author

ADjenkov commented Jan 4, 2019

test ngapps

@dtopuzov
Copy link
Copy Markdown
Contributor

dtopuzov commented Jan 4, 2019

test --ignore templates api23 ios12

@ADjenkov ADjenkov merged commit 984f162 into master Jan 4, 2019
@ghost ghost removed bug in progress labels Jan 4, 2019
@ADjenkov 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
Copy link
Copy Markdown

lock Bot commented Jan 4, 2020

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 Jan 4, 2020
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.