Merge "Making sure QS are visible when expanding shade with heads-up notification" into tm-qpr-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
dae823a3b6
@@ -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;
|
||||||
|
|||||||
Reference in New Issue
Block a user