Fixed
Status Update
Comments
ap...@google.com <ap...@google.com> #2
Nested scrolling works partially (as per http://b/122818889 ). Let's discuss if we need full support and if so make sure it works.
jb...@google.com <jb...@google.com> #3
Hi!
What is 'partially' exactly?
How do I see it?
Thanks!
What is 'partially' exactly?
How do I see it?
Thanks!
Description
With the currently implementation of
visibleTransitionsInProgress
in possible that there is a scenario where you can have duplicate entries in the list and cause the wrong back stack entry to be composed.This is because we first add all of the
STARTED
entries to the lists and then we add aLifecycleObserver
than then re-adds entries when they go throughON_START
.While we could change the list to a
MutableSet
to avoid this, we would then lose the snapshot functionality ofmutableStateListOf()
and we want to still recompose in response to adding and removing entries from the list. So we should just treat the list like it is a set manually instead.