Merge "Only set arrow color once when below icon." into sc-v2-dev am: 3fca6717ed

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15936545

Change-Id: I6c1d7d2daa9cc78f3c9ca3b57234d6e0d02aaa15
This commit is contained in:
Brian Isganitis
2021-09-29 19:59:53 +00:00
committed by Automerger Merge Worker

View File

@@ -270,6 +270,15 @@ public abstract class ArrowPopup<T extends Context & ActivityContext>
backgroundColor = colors[numVisibleChild % colors.length];
}
if (!ENABLE_LOCAL_COLOR_POPUPS.get()) {
// Arrow color matches the first child or the last child.
if (!mIsAboveIcon && numVisibleChild == 0 && viewGroup == this) {
mArrowColor = backgroundColor;
} else if (mIsAboveIcon) {
mArrowColor = backgroundColor;
}
}
if (view instanceof ViewGroup && mIterateChildrenTag.equals(view.getTag())) {
assignMarginsAndBackgrounds((ViewGroup) view, backgroundColor);
numVisibleChild++;
@@ -293,12 +302,6 @@ public abstract class ArrowPopup<T extends Context & ActivityContext>
if (!ENABLE_LOCAL_COLOR_POPUPS.get()) {
setChildColor(view, backgroundColor, colorAnimator);
// Arrow color matches the first child or the last child.
if (!mIsAboveIcon && numVisibleChild == 0) {
mArrowColor = backgroundColor;
} else if (mIsAboveIcon) {
mArrowColor = backgroundColor;
}
}
numVisibleChild++;