From cfc91cf2648cdaa2268e70ebfeed421ed86a6221 Mon Sep 17 00:00:00 2001 From: Selim Cinek Date: Wed, 10 Sep 2014 18:52:08 +0200 Subject: [PATCH] Fixed a bug where the drag down animation was behaving wrong Since we actually only need to animate the yCoordinate in the drag down motion, some clipping artefacts were left in certain cases. Bug: 17345758 Change-Id: I659a64b7ae5c7abcfa04c20347ec56a2d5f4f75c --- .../android/systemui/statusbar/stack/StackStateAnimator.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/stack/StackStateAnimator.java b/packages/SystemUI/src/com/android/systemui/statusbar/stack/StackStateAnimator.java index ece82a1cc8d46..a69390ef0f898 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/stack/StackStateAnimator.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/stack/StackStateAnimator.java @@ -166,7 +166,7 @@ public class StackStateAnimator { boolean hasDelays = mAnimationFilter.hasDelays; boolean isDelayRelevant = yTranslationChanging || zTranslationChanging || scaleChanging || alphaChanging || heightChanging || topInsetChanging; - boolean noAnimation = wasAdded && !mAnimationFilter.hasGoToFullShadeEvent; + boolean noAnimation = wasAdded; long delay = 0; long duration = mCurrentLength; if (hasDelays && isDelayRelevant || wasAdded) { @@ -184,7 +184,7 @@ public class StackStateAnimator { // start translationY animation if (yTranslationChanging) { - if (noAnimation) { + if (noAnimation && !mAnimationFilter.hasGoToFullShadeEvent) { child.setTranslationY(viewState.yTranslation); } else { startYTranslationAnimation(child, viewState, duration, delay);