Fixed
Status Update
Comments
il...@google.com <il...@google.com>
ap...@google.com <ap...@google.com> #2
Hi. Thanks for reporting this. Fixed in alpha-04
jb...@google.com <jb...@google.com> #3
Project: platform/frameworks/support
Branch: androidx-main
commit e782987543a9f8ccd485e970ddc74564b24378db
Author: Vighnesh Raut <vighnesh.raut13@gmail.com>
Date: Mon Jan 02 15:27:40 2023
fix: tab row crashes when only 1 tab is added
Bug: b/264018028
Test: Added unit test
Change-Id: I6381dbac304fc1d69d3708c6655f8b595668e93f
M tv/tv-material/src/androidTest/java/androidx/tv/material/TabRowTest.kt
M tv/tv-material/src/main/java/androidx/tv/material/TabRow.kt
https://android-review.googlesource.com/2373449
Branch: androidx-main
commit e782987543a9f8ccd485e970ddc74564b24378db
Author: Vighnesh Raut <vighnesh.raut13@gmail.com>
Date: Mon Jan 02 15:27:40 2023
fix: tab row crashes when only 1 tab is added
Bug:
Test: Added unit test
Change-Id: I6381dbac304fc1d69d3708c6655f8b595668e93f
M tv/tv-material/src/androidTest/java/androidx/tv/material/TabRowTest.kt
M tv/tv-material/src/main/java/androidx/tv/material/TabRow.kt
an...@google.com <an...@google.com> #4
deleted
il...@google.com <il...@google.com> #5
The following release(s) address this bug.It is possible this bug has only been partially addressed:
androidx.tv:tv-material:1.0.0-alpha04
Description
Component used: Navigation Version used:2.3.0-alpha05
Devices/Android versions reproduced on: Emulator, Android 11 DP3
I'm trying out returning results from dialogFragments with the SavedStateHandle you provide in this latest versions. it works well, taking into account the "Additional Considerations" section advice, of observing the NavBackStackEntry lifecycle, so the weird stuff that happens of having the fragment and the dialog being resumed at the same time doesn't affect when we receive the result.
But, process death is problematic, first I encountered https://issuetracker.google.com/154405007
It's easy to fix once you know what's happening. But now the problem I'm seeing is that the lifecycle events do not work in the same way when process death has occurred and backstack state is restored.
Normally, what I see is this, on the currentBackStackEntry: onCreate -> onStart -> onResume -> (we open the dialog) -> onPause ... and then onResume again when we come back from the dialog. Everything is cool here.
But after process death, this happens: onCreate -> onStart -> onResume We never hit onPause, and so we never "come back" to onResume after dismissing the dialog. We will never receiver the ON_RESUME event again to get the result.
I'm not sure if this is a bug, or just a consequence of how the state restoration is done on the backstack, but it pretty much means that thishttps://developer.android.com/guide/navigation/navigation-programmatic#additional_considerations , won't always work.
I attach a minimal repo to reproduce this, just click on the dialog button, and on receiving a result on the first fragment a toast will appear. Now, if you go to dialog once again, leave the app, and trigger process death and return to the app, it won't work.https://github.com/MrMPN/NavEntryLifecycleMinRepo
It's not a common case I think, but given the info on the documentation, it was unexpected to see it fail like this.