Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a4723845 authored by Luca Zuccarini's avatar Luca Zuccarini
Browse files

Only enforce return animations when going to Launcher. 

This was discussed with UX, and we came to the conclusion that
cross-task animation should use the default as it ensures a better
visual cue that the closing window is going to the background.

Bug: 323863002
Flag: com.android.systemui.shared.return_animation_framework_library
Flag: com.android.systemui.shared.return_animation_framework_long_lived
Test: manual
Change-Id: Idaa649a3eacfb0d045e4163b02312cbcffcd91d7
parent 892bbff3

packages/SystemUI/animation/src/com/android/systemui/animation/ActivityTransitionAnimator.kt

+7 −1
Original line number Diff line number Diff line
@@ -702,6 +702,8 @@ constructor( 
            object : Controller by controller {

                override val isLaunching: Boolean = false

            }

        // Cross-task close transitions should not use this animation, so we only register it for

        // when the opening window is Launcher.

        val returnFilter =

            TransitionFilter().apply {

                mRequirements =
@@ -710,7 +712,11 @@ constructor( 
                            mActivityType = WindowConfiguration.ACTIVITY_TYPE_STANDARD

                            mModes = intArrayOf(TRANSIT_CLOSE, TRANSIT_TO_BACK)

                            mTopActivity = component

                        }

                        },

                        TransitionFilter.Requirement().apply {

                            mActivityType = WindowConfiguration.ACTIVITY_TYPE_HOME

                            mModes = intArrayOf(TRANSIT_OPEN, TRANSIT_TO_FRONT)

                        },

                    )

            }

        val returnRemoteTransition =