Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Avoid weird expand animation when a notification is updated

Browse Source 
Whenever a notification was updated and it was expanded, the
animation would look incredibly weird. This is now fixed.

Change-Id: Iaeac6f71c3771c4782e751be50b54b62d08b643e
Fixes: 30693397
This commit is contained in:
Selim Cinek
2016-08-10 13:02:38 -07:00
parent 2dabc8286b
commit bb42b7dd48
2 changed files with 3 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
packages/SystemUI/src/com/android/systemui/statusbar/ExpandableNotificationRow.java
View File

@@ -507,7 +507,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView {
int intrinsicHeight = getIntrinsicHeight();
mIsPinned = pinned;
if (intrinsicHeight != getIntrinsicHeight()) {
notifyHeightChanged(false);
notifyHeightChanged(false /* needsAnimation */);
}
if (pinned) {
setIconAnimationRunning(true);
@@ -840,8 +840,6 @@ public class ExpandableNotificationRow extends ActivatableNotificationView {
}
public void resetHeight() {
mMaxExpandHeight = 0;
mHeadsUpHeight = 0;
onHeightReset();
requestLayout();
}
@@ -1290,7 +1288,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView {
}
mHeadsUpHeight = headsUpChild.getHeight();
if (intrinsicBefore != getIntrinsicHeight()) {
notifyHeightChanged(false /* needsAnimation */);
notifyHeightChanged(true /* needsAnimation */);
}
}
@@ -1398,7 +1396,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView {
if (isChildInGroup()) {
mGroupManager.setGroupExpanded(mStatusBarNotification, true);
}
notifyHeightChanged(false);
notifyHeightChanged(false /* needsAnimation */);
}
public void setChildrenExpanded(boolean expanded, boolean animate) {

8
packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java
View File

@@ -216,7 +216,6 @@ public class NotificationStackScrollLayout extends ViewGroup
private float mTopPaddingOverflow;
private boolean mDontReportNextOverScroll;
private boolean mDontClampNextScroll;
private boolean mRequestViewResizeAnimationOnLayout;
private boolean mNeedViewResizeAnimation;
private View mExpandedGroupView;
private boolean mEverythingNeedsAnimation;
@@ -515,10 +514,6 @@ public class NotificationStackScrollLayout extends ViewGroup
setMaxLayoutHeight(getHeight());
updateContentHeight();
clampScrollPosition();
if (mRequestViewResizeAnimationOnLayout) {
requestAnimationOnViewResize(null);
mRequestViewResizeAnimationOnLayout = false;
}
requestChildrenUpdate();
updateFirstAndLastBackgroundViews();
}
@@ -3100,9 +3095,6 @@ public class NotificationStackScrollLayout extends ViewGroup
@Override
public void onReset(ExpandableView view) {
if (mIsExpanded && mAnimationsEnabled) {
mRequestViewResizeAnimationOnLayout = true;
}
updateAnimationState(view);
updateChronometerForChild(view);
}