Fixed
Status Update
No update yet.
Comments
jn...@google.com
Hi - thanks for raising this, I'll dig into our SwipeDismissableNavHost and try to find out why we didn't support the back button already. Will report back on this thread.
jn...@google.com
An early draft of SwipeDismissableNavHost did implement back button support, but it was thought not to be required - we will submit a change to add the support back again.
jn...@google.com
jn...@google.com
ap...@google.com
Project: platform/frameworks/support
Branch: androidx-main
commit 78d498661680a95d2705d5e6dbb1c0438c50fa30
Author: stevebower <stevebower@google.com>
Date: Tue Dec 14 13:42:48 2021
Support back button in SwipeDismissableNavHost.
Added back button handling for navigation to
SwipeDismissableNavHost, using the implementation
from Compose Material NavHost. This enables use
of the emulator's back button for local testing,
as well as in any devices that support a hardware back button.
Test: Run tests for wear.compose.navigation.
Bug: 210205624
Relnote: "We have added a back button handler to
SwipeDismissableNavHost, so that pressing
back navigates to the previous level
in the navigation hierarchy."
Change-Id: I5b08689b1651c1aae5f038f058956007df398909
M wear/compose/compose-navigation/build.gradle
M wear/compose/compose-navigation/src/androidTest/kotlin/androidx/wear/compose/navigation/SwipeDismissableNavHostTest.kt
M wear/compose/compose-navigation/src/main/java/androidx/wear/compose/navigation/SwipeDismissableNavHost.kt
https://android-review.googlesource.com/1922537
Branch: androidx-main
commit 78d498661680a95d2705d5e6dbb1c0438c50fa30
Author: stevebower <stevebower@google.com>
Date: Tue Dec 14 13:42:48 2021
Support back button in SwipeDismissableNavHost.
Added back button handling for navigation to
SwipeDismissableNavHost, using the implementation
from Compose Material NavHost. This enables use
of the emulator's back button for local testing,
as well as in any devices that support a hardware back button.
Test: Run tests for wear.compose.navigation.
Bug: 210205624
Relnote: "We have added a back button handler to
SwipeDismissableNavHost, so that pressing
back navigates to the previous level
in the navigation hierarchy."
Change-Id: I5b08689b1651c1aae5f038f058956007df398909
M wear/compose/compose-navigation/build.gradle
M wear/compose/compose-navigation/src/androidTest/kotlin/androidx/wear/compose/navigation/SwipeDismissableNavHostTest.kt
M wear/compose/compose-navigation/src/main/java/androidx/wear/compose/navigation/SwipeDismissableNavHost.kt
jn...@google.com
So we have managed to isolate and reproduce the problem on the emulator and add some targeted tests.
[Deleted User] <[Deleted User]>
Comment has been deleted.
[Deleted User] <[Deleted User]>
Thanks for the prompt response, so glad that root cause is found! :)
I would work-around the problem for now, so please let me know if the bug fix is released. Thanks again!
I would work-around the problem for now, so please let me know if the bug fix is released. Thanks again!
jn...@google.com
Release in Compose for Wear OS rc01
jn...@google.com
je...@tagheuer.com
Comment has been deleted.
Description
Version used: 1.0.0-beta02
Devices/Android versions reproduced on:
- Can be 100% reproduced on Samsung SM-R870, SM-R865F (both are Android 11, Wear 3.2, One UI 4.0)
- Failed to reproduce on Wear OS Simulator (API 30, Wear 3.2)
Description:
I found sometimes ScalingLazyColumn wouldn't display items at first until I scrolled the view. (My displayed items are not dynamically changed by recompose. The only thing I do is scrolling.)
And I also found that (1) if I add another item to ScalingLazyColumn and rebuild the app, or (2) if I add some vertical padding to one of the item,
items could be display successfully.
So I guess it might be related to the item height or item count... I'm not sure.
- Code:
(With the code above, the issue could be 100% reproduce on Samsung SM-R870, SM-R865F)