Merge "Making sure QS are visible when expanding shade with heads-up notification" into tm-qpr-dev

This commit is contained in:
TreeHugger Robot
2022-12-05 23:20:27 +00:00
committed by Android (Google) Code Review

View File

@@ -465,7 +465,11 @@ public final class NotificationPanelViewController implements Dumpable {
private boolean mQsTouchAboveFalsingThreshold; private boolean mQsTouchAboveFalsingThreshold;
private int mQsFalsingThreshold; private int mQsFalsingThreshold;
/** Indicates drag starting height when swiping down or up on heads-up notifications */ /**
* Indicates drag starting height when swiping down or up on heads-up notifications.
* This usually serves as a threshold from when shade expansion should really start. Otherwise
* this value would be height of shade and it will be immediately expanded to some extent.
*/
private int mHeadsUpStartHeight; private int mHeadsUpStartHeight;
private HeadsUpTouchHelper mHeadsUpTouchHelper; private HeadsUpTouchHelper mHeadsUpTouchHelper;
private boolean mListenForHeadsUp; private boolean mListenForHeadsUp;
@@ -3412,9 +3416,12 @@ public final class NotificationPanelViewController implements Dumpable {
&& mQsExpansionAnimator == null && !mQsExpansionFromOverscroll; && mQsExpansionAnimator == null && !mQsExpansionFromOverscroll;
boolean goingBetweenClosedShadeAndExpandedQs = boolean goingBetweenClosedShadeAndExpandedQs =
mQsExpandImmediate || collapsingShadeFromExpandedQs; mQsExpandImmediate || collapsingShadeFromExpandedQs;
// we don't want to update QS expansion when HUN is visible because then the whole shade is // in split shade we react when HUN is visible only if shade height is over HUN start
// initially hidden, even though it has non-zero height // height - which means user is swiping down. Otherwise shade QS will either not show at all
if (goingBetweenClosedShadeAndExpandedQs && !mHeadsUpManager.isTrackingHeadsUp()) { // with HUN movement or it will blink when touching HUN initially
boolean qsShouldExpandWithHeadsUp = !mSplitShadeEnabled
|| (!mHeadsUpManager.isTrackingHeadsUp() || expandedHeight > mHeadsUpStartHeight);
if (goingBetweenClosedShadeAndExpandedQs && qsShouldExpandWithHeadsUp) {
float qsExpansionFraction; float qsExpansionFraction;
if (mSplitShadeEnabled) { if (mSplitShadeEnabled) {
qsExpansionFraction = 1; qsExpansionFraction = 1;