Merge "Showing the notification icon properly now when the pulse is suppressed" into qt-dev

This commit is contained in:
TreeHugger Robot
2019-07-29 20:33:42 +00:00
committed by Android (Google) Code Review
3 changed files with 8 additions and 5 deletions

View File

@@ -512,7 +512,7 @@ public class NotificationShelf extends ActivatableNotificationView implements
float viewEnd = row.getTranslationY() + row.getActualHeight(); float viewEnd = row.getTranslationY() + row.getActualHeight();
boolean isPinned = (row.isPinned() || row.isHeadsUpAnimatingAway()) boolean isPinned = (row.isPinned() || row.isHeadsUpAnimatingAway())
&& !mAmbientState.isDozingAndNotPulsing(row); && !mAmbientState.isDozingAndNotPulsing(row);
boolean shouldClipOwnTop = row.showingAmbientPulsing() boolean shouldClipOwnTop = row.showingAmbientPulsing() && !mAmbientState.isFullyDark()
|| (mAmbientState.isPulseExpanding() && childIndex == 0); || (mAmbientState.isPulseExpanding() && childIndex == 0);
if (viewEnd > notificationClipEnd && !shouldClipOwnTop if (viewEnd > notificationClipEnd && !shouldClipOwnTop
&& (mAmbientState.isShadeExpanded() || !isPinned)) { && (mAmbientState.isShadeExpanded() || !isPinned)) {
@@ -752,8 +752,9 @@ public class NotificationShelf extends ActivatableNotificationView implements
iconState.scaleY = 1.0f; iconState.scaleY = 1.0f;
iconState.hidden = false; iconState.hidden = false;
} }
if (row.isAboveShelf() || (!row.isInShelf() && (isLastChild && row.areGutsExposed() if ((row.isAboveShelf() || (!row.isInShelf() && (isLastChild && row.areGutsExposed()
|| row.getTranslationZ() > mAmbientState.getBaseZHeight()))) { || row.getTranslationZ() > mAmbientState.getBaseZHeight())))
&& !mAmbientState.isFullyDark()) {
iconState.hidden = true; iconState.hidden = true;
} }
int backgroundColor = getBackgroundColorWithoutTint(); int backgroundColor = getBackgroundColorWithoutTint();

View File

@@ -1355,7 +1355,7 @@ public class NotificationStackScrollLayout extends ViewGroup implements ScrollAd
mIsClipped = clipped; mIsClipped = clipped;
} }
if (!mAmbientPulseManager.hasNotifications() && mAmbientState.isFullyDark()) { if (!mPulsing && mAmbientState.isFullyDark()) {
setClipBounds(null); setClipBounds(null);
} else if (mAmbientState.isDarkAtAll()) { } else if (mAmbientState.isDarkAtAll()) {
clipToOutline = true; clipToOutline = true;
@@ -5169,6 +5169,7 @@ public class NotificationStackScrollLayout extends ViewGroup implements ScrollAd
return; return;
} }
mPulsing = pulsing; mPulsing = pulsing;
updateClipping();
mAmbientState.setPulsing(pulsing); mAmbientState.setPulsing(pulsing);
mSwipeHelper.setPulsing(pulsing); mSwipeHelper.setPulsing(pulsing);
updateNotificationAnimationStates(); updateNotificationAnimationStates();

View File

@@ -501,7 +501,8 @@ public class StackScrollAlgorithm {
continue; continue;
} }
ExpandableNotificationRow row = (ExpandableNotificationRow) child; ExpandableNotificationRow row = (ExpandableNotificationRow) child;
if (!row.showingAmbientPulsing() || (i == 0 && ambientState.isPulseExpanding())) { if (!row.showingAmbientPulsing() || ambientState.isFullyDark()
|| (i == 0 && ambientState.isPulseExpanding())) {
continue; continue;
} }
ExpandableViewState viewState = row.getViewState(); ExpandableViewState viewState = row.getViewState();