From 56f542f249cd416592516ed89fda0f401482a773 Mon Sep 17 00:00:00 2001 From: Selim Cinek Date: Fri, 3 Feb 2017 11:22:25 -0800 Subject: [PATCH] Fixed a bug where the notification icons wouldn't match If there was just one ambient notification available, we Would suck in one icon too much when expanded. Test: add 1 high and 1 min priority notification, observe icons Change-Id: I043011c6d99c49b9764017565ee48349fce920eb --- .../systemui/statusbar/phone/NotificationIconContainer.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationIconContainer.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationIconContainer.java index c25a45cc37e0f..4c18512059a28 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationIconContainer.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationIconContainer.java @@ -255,7 +255,7 @@ public class NotificationIconContainer extends AlphaOptimizedFrameLayout { iconState.visibleState = StatusBarIconView.STATE_ICON; if (firstOverflowIndex == -1 && (isAmbient || (translationX >= (noOverflowAfter ? layoutEnd - mIconSize : overflowStart)))) { - firstOverflowIndex = noOverflowAfter ? i - 1 : i; + firstOverflowIndex = noOverflowAfter && !isAmbient ? i - 1 : i; int totalDotLength = mStaticDotRadius * 6 + 2 * mDotPadding; visualOverflowStart = overflowStart + mIconSize * (1 + OVERFLOW_EARLY_AMOUNT) - totalDotLength / 2