summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Nicolo' Mazzucato <nicomazz@google.com> 2025-02-05 13:30:23 +0000
committer Nicolo' Mazzucato <nicomazz@google.com> 2025-02-05 16:53:03 +0000
commit65d2cf71f60100a18eb06ea070157b1719b44a8d (patch)
tree5f49d5657910162022bd2437a24c873a3879b86b
parent7b27cbe7dc466b14e111b4aff5f26e43143c90b7 (diff)
Do not expand/collapse shade after display move when flexiglass is off
Expanding/collapsing the shade is not supported using ShadeInteractor for the legacy shade. This cl removes that behaviour while SceneContainerFlag is off, making the shade move work also for the legacy shade (and allowing us to turn on the flag in trunk staging and write e2e tests already) Bug: 362719719 Test: ShadeExpandedStateInteractorTest Flag: com.android.systemui.shade_window_goes_around Change-Id: I2f38f530802c50304bd7d4accf058c591bfa6930
-rw-r--r--packages/SystemUI/src/com/android/systemui/shade/domain/interactor/ShadeExpandedStateInteractor.kt24
1 files changed, 16 insertions, 8 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/shade/domain/interactor/ShadeExpandedStateInteractor.kt b/packages/SystemUI/src/com/android/systemui/shade/domain/interactor/ShadeExpandedStateInteractor.kt
index 8b110e3ef792..c60a67f8c8d0 100644
--- a/packages/SystemUI/src/com/android/systemui/shade/domain/interactor/ShadeExpandedStateInteractor.kt
+++ b/packages/SystemUI/src/com/android/systemui/shade/domain/interactor/ShadeExpandedStateInteractor.kt
@@ -110,13 +110,17 @@ constructor(
@Background private val bgContext: CoroutineContext,
) : ShadeElement() {
override suspend fun expand(reason: String) {
- shadeInteractor.expandNotificationsShade(reason)
- shadeInteractor.shadeExpansion.waitUntil(1f, bgContext)
+ if (SceneContainerFlag.isEnabled) {
+ shadeInteractor.expandNotificationsShade(reason)
+ shadeInteractor.shadeExpansion.waitUntil(1f, bgContext)
+ }
}
override suspend fun collapse(reason: String) {
- shadeInteractor.collapseNotificationsShade(reason)
- shadeInteractor.shadeExpansion.waitUntil(0f, bgContext)
+ if (SceneContainerFlag.isEnabled) {
+ shadeInteractor.collapseNotificationsShade(reason)
+ shadeInteractor.shadeExpansion.waitUntil(0f, bgContext)
+ }
}
}
@@ -128,12 +132,16 @@ constructor(
@Background private val bgContext: CoroutineContext,
) : ShadeElement() {
override suspend fun expand(reason: String) {
- shadeInteractor.expandQuickSettingsShade(reason)
- shadeInteractor.qsExpansion.waitUntil(1f, bgContext)
+ if (SceneContainerFlag.isEnabled) {
+ shadeInteractor.expandQuickSettingsShade(reason)
+ shadeInteractor.qsExpansion.waitUntil(1f, bgContext)
+ }
}
override suspend fun collapse(reason: String) {
- shadeInteractor.collapseQuickSettingsShade(reason)
- shadeInteractor.qsExpansion.waitUntil(0f, bgContext)
+ if (SceneContainerFlag.isEnabled) {
+ shadeInteractor.collapseQuickSettingsShade(reason)
+ shadeInteractor.qsExpansion.waitUntil(0f, bgContext)
+ }
}
}