From 7cb49d6f40955652950409b4feead43e58f68ca5 Mon Sep 17 00:00:00 2001 From: Selim Cinek Date: Fri, 26 May 2017 13:48:47 -0700 Subject: Fixed an issue where the shade would jump When collapsing directly from the expanded quicksettings the shade would jump when there were no notifications. Test: manual Merged-In: I9a85cd57e2f96970ad4ea78a8c679145b58d563a Change-Id: I9a85cd57e2f96970ad4ea78a8c679145b58d563a Fixes: 35996906 --- .../systemui/statusbar/stack/NotificationStackScrollLayout.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java b/packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java index 61fed2dfdcfb..d176b66caaec 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java @@ -815,7 +815,8 @@ public class NotificationStackScrollLayout extends ViewGroup */ private float getAppearEndPosition() { int appearPosition; - if (mEmptyShadeView.getVisibility() == GONE) { + int notGoneChildCount = getNotGoneChildCount(); + if (mEmptyShadeView.getVisibility() == GONE && notGoneChildCount != 0) { int minNotificationsForShelf = 1; if (mTrackingHeadsUp || mHeadsUpManager.hasPinnedHeadsUp()) { appearPosition = mHeadsUpManager.getTopHeadsUpPinnedHeight(); @@ -823,7 +824,7 @@ public class NotificationStackScrollLayout extends ViewGroup } else { appearPosition = 0; } - if (getNotGoneChildCount() >= minNotificationsForShelf) { + if (notGoneChildCount >= minNotificationsForShelf) { appearPosition += mShelf.getIntrinsicHeight(); } } else { -- cgit v1.2.3-59-g8ed1b