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

Commit d95569b4 authored by Johannes Gallmann's avatar Johannes Gallmann
Browse files

Revert "Revert "Fix notification group dismissal fade animation""

This reverts commit eb145298.

Reason for revert: Reverting the "Fix notification group dismissal fade animation" CL did not cause the test metric to recover. Apparently, another CL caused the metric to increase. Therefore, it should be save to merge this change back in.

Bug: 253532643
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.
Change-Id: Iadfa1db6630bf01ca3a36977c41366af77e4b263
parent ec9b5b0d
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -1526,7 +1526,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView
            l.setAlpha(alpha);
        }
        if (mChildrenContainer != null) {
            mChildrenContainer.setAlpha(alpha);
            mChildrenContainer.setContentAlpha(alpha);
        }
    }

+14 −0
Original line number Diff line number Diff line
@@ -494,6 +494,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
     */