Fixed
Status Update
Comments
ap...@google.com <ap...@google.com> #2
Project: platform/frameworks/support
Branch: androidx-master-dev
commit 3f9632ad04cacaf1fd118b48c49238e2535df018
Author: Jeremy Woods <jbwoods@google.com>
Date: Thu Jul 18 15:15:07 2019
Ensure postponedEnter with duration only starts once
Right now if the postponedEnterTransition(duration, TimeUnit) is called
twice, startPostponedEnterTransition is also called twice. The proper
behavior is for the first Runnable to be replaced if a second is called
before the duration time runs out.
This change maintains the current Runnable in the Fragment's
AnimationInfo and if postponedEnterTransition is called while another
runnable already exists that Runnable is removed from the handler.
Test: Added Test
BUG: 137797118
Change-Id: Iecd55a1267e7613d025e0b95393996ad53dcace3
M fragment/fragment/src/androidTest/java/androidx/fragment/app/PostponedTransitionTest.kt
M fragment/fragment/src/main/java/androidx/fragment/app/Fragment.java
https://android-review.googlesource.com/1087225
https://goto.google.com/android-sha1/3f9632ad04cacaf1fd118b48c49238e2535df018
Branch: androidx-master-dev
commit 3f9632ad04cacaf1fd118b48c49238e2535df018
Author: Jeremy Woods <jbwoods@google.com>
Date: Thu Jul 18 15:15:07 2019
Ensure postponedEnter with duration only starts once
Right now if the postponedEnterTransition(duration, TimeUnit) is called
twice, startPostponedEnterTransition is also called twice. The proper
behavior is for the first Runnable to be replaced if a second is called
before the duration time runs out.
This change maintains the current Runnable in the Fragment's
AnimationInfo and if postponedEnterTransition is called while another
runnable already exists that Runnable is removed from the handler.
Test: Added Test
BUG: 137797118
Change-Id: Iecd55a1267e7613d025e0b95393996ad53dcace3
M fragment/fragment/src/androidTest/java/androidx/fragment/app/PostponedTransitionTest.kt
M fragment/fragment/src/main/java/androidx/fragment/app/Fragment.java
an...@google.com <an...@google.com> #3
il...@google.com <il...@google.com> #4
This is fixed and will be part of Fragments 1.1.0-rc03
Description
The Fragment should maintain the Runnable so it can be removed later.