From 1e34f653953518665e3059e73c42d6a2dbdbed06 Mon Sep 17 00:00:00 2001 From: Brian Isganitis Date: Tue, 28 Sep 2021 19:36:59 -0400 Subject: [PATCH] Update arrow color after all notifications dismissed. assignMarginsAndBackgrounds updates determines the new arrow color, but it doesn't actually update it. That is normally done in updateArrowColor. Test: Icon menu arrow color changes to content above it after notification dots dismissed. Fix: 201458705 Change-Id: I5ee40b420b03c29c197983af5968d4f2902bc59b --- src/com/android/launcher3/popup/ArrowPopup.java | 2 +- src/com/android/launcher3/popup/PopupContainerWithArrow.java | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/com/android/launcher3/popup/ArrowPopup.java b/src/com/android/launcher3/popup/ArrowPopup.java index 117ae4212b..1a5d1410fa 100644 --- a/src/com/android/launcher3/popup/ArrowPopup.java +++ b/src/com/android/launcher3/popup/ArrowPopup.java @@ -495,7 +495,7 @@ public abstract class ArrowPopup mArrow.setPivotY(mIsAboveIcon ? mArrowHeight : 0); } - private void updateArrowColor() { + protected void updateArrowColor() { if (!Gravity.isVertical(mGravity)) { mArrow.setBackground(new RoundedArrowDrawable( mArrowWidth, mArrowHeight, mArrowPointRadius, diff --git a/src/com/android/launcher3/popup/PopupContainerWithArrow.java b/src/com/android/launcher3/popup/PopupContainerWithArrow.java index e340b21d53..65dd8eaee1 100644 --- a/src/com/android/launcher3/popup/PopupContainerWithArrow.java +++ b/src/com/android/launcher3/popup/PopupContainerWithArrow.java @@ -593,6 +593,7 @@ public class PopupContainerWithArrow> mNotificationContainer.setVisibility(GONE); updateHiddenShortcuts(); assignMarginsAndBackgrounds(PopupContainerWithArrow.this); + updateArrowColor(); } else { mNotificationContainer.trimNotifications( NotificationKeyData.extractKeysOnly(dotInfo.getNotificationKeys()));