Commit d8018445 authored Feb 10, 2022 by Michal Brzezinski

Fixing not responsive tiles in split shade

Some tiles (beyond QQS tiles) were not responsive because when QSPanelController is about to start listening, it also checks if mQsExpanded is set to true. mQsExpanded is passed from NotificationPanelViewController and it wasn't set to true because function setting it is never called in split shade.
This change is hacky but probably safer way to solve it by changing condition for listening on changes.

Bug: 218268829
Test: open split shade and tap night mode tile a few times - the tile must be positioned beyond first 6 tiles
Change-Id: Id6ad9f199964dd416a52c169a3e30713389479cc

parent dbad339f

packages/SystemUI/src/com/android/systemui/qs/QSPanelController.java

+3 −1

Original line number	Diff line number	Diff line
		@@ -187,7 +187,9 @@ public class QSPanelController extends QSPanelControllerBase<QSPanel> {

		/** */
		public void setListening(boolean listening, boolean expanded) {
		setListening(listening && expanded);
		// TODO(218268829): checking for split shade is workaround but when proper fix lands
		// "\|\| mShouldUseSplitNotificationShade" should be removed
		setListening(listening && (expanded \|\| mShouldUseSplitNotificationShade));
		if (mView.isListening()) {
		refreshAllTiles();
		}