From ed34bfaf1bf579b65f3564bd7366c837d92f1a8b Mon Sep 17 00:00:00 2001 From: Fabian Kozynski Date: Thu, 17 Jun 2021 13:18:28 -0400 Subject: [PATCH] Remove marquee for QQS Tiles Test: manual Fixes: 188009384 Change-Id: Ied010001bf95648047c77d1014b0bc29eb6e71d9 --- .../com/android/systemui/qs/QuickQSPanel.java | 19 ------------------- .../systemui/qs/tileimpl/QSTileViewImpl.kt | 3 +++ 2 files changed, 3 insertions(+), 19 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/qs/QuickQSPanel.java b/packages/SystemUI/src/com/android/systemui/qs/QuickQSPanel.java index e24acf27c9735..659475d192779 100644 --- a/packages/SystemUI/src/com/android/systemui/qs/QuickQSPanel.java +++ b/packages/SystemUI/src/com/android/systemui/qs/QuickQSPanel.java @@ -228,24 +228,5 @@ public class QuickQSPanel extends QSPanel { } } } - - @Override - public void setExpansion(float expansion) { - if (expansion > 0f && expansion < 1f) { - return; - } - boolean selected = expansion == 0f; - // Expansion == 0f is when QQS is fully showing (as opposed to 1f, which is QS). At this - // point we want them to be selected so the tiles will marquee (but not at other points - // of expansion. - // We set it as not important while we change this, so setting each tile as selected - // will not cause them to announce themselves until the user has actually selected the - // item. - setImportantForAccessibility(View.IMPORTANT_FOR_ACCESSIBILITY_NO_HIDE_DESCENDANTS); - for (int i = 0; i < getChildCount(); i++) { - getChildAt(i).setSelected(selected); - } - setImportantForAccessibility(View.IMPORTANT_FOR_ACCESSIBILITY_AUTO); - } } } diff --git a/packages/SystemUI/src/com/android/systemui/qs/tileimpl/QSTileViewImpl.kt b/packages/SystemUI/src/com/android/systemui/qs/tileimpl/QSTileViewImpl.kt index 8aed0a88fa35d..ca782e56de430 100644 --- a/packages/SystemUI/src/com/android/systemui/qs/tileimpl/QSTileViewImpl.kt +++ b/packages/SystemUI/src/com/android/systemui/qs/tileimpl/QSTileViewImpl.kt @@ -190,6 +190,9 @@ open class QSTileViewImpl @JvmOverloads constructor( if (collapsed) { labelContainer.ignoreLastView = true secondaryLabel.alpha = 0f + // Do not marque in QQS + label.ellipsize = TextUtils.TruncateAt.END + secondaryLabel.ellipsize = TextUtils.TruncateAt.END } setLabelColor(getLabelColorForState(QSTile.State.DEFAULT_STATE)) setSecondaryLabelColor(getSecondaryLabelColorForState(QSTile.State.DEFAULT_STATE))