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

Fix notification group dismissal fade animation

Browse Source 
When dismissing a notification group, only the content should be faded but not the background.

Test: Manual, i.e. posting a Standard notification and an Inbox notification from the Notify2-RVC App (as well as the Notify2-SC app), then dismissing both notifications together by swiping the group header.
Bug: 202138492
Change-Id: If7c9eb99ee6a67566b77520c69138e0d5b543381
This commit is contained in:
Johannes Gallmann
2022-10-10 10:19:39 +00:00
parent c5bfa99295
commit 01397a7825
2 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/SystemUI/src/com/android/systemui/statusbar/notification/row/ExpandableNotificationRow.java
View File

@@ -1487,7 +1487,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView
l.setAlpha(alpha);
}
if (mChildrenContainer != null) {
mChildrenContainer.setAlpha(alpha);
mChildrenContainer.setContentAlpha(alpha);
}
}

14
packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationChildrenContainer.java
View File

@@ -461,6 +461,20 @@ public class NotificationChildrenContainer extends ViewGroup
return mAttachedChildren;
}
/**
* Sets the alpha on the content, while leaving the background of the container itself as is.
*
* @param alpha alpha value to apply to the content
*/
public void setContentAlpha(float alpha) {
for (int i = 0; i < mNotificationHeader.getChildCount(); i++) {
mNotificationHeader.getChildAt(i).setAlpha(alpha);
}
for (ExpandableNotificationRow child : getAttachedChildren()) {
child.setContentAlpha(alpha);
}
}
/** To be called any time the rows have been updated */
public void updateExpansionStates() {
if (mChildrenExpanded || mUserLocked) {