From 93985b2a6c87786a9bcec270fd683763ae168506 Mon Sep 17 00:00:00 2001
From: MM20 <15646950+MM2-0@users.noreply.github.com>
Date: Fri, 20 Jan 2023 11:45:08 +0100
Subject: [PATCH] Gesture settings: don't show permission banners for actions
 that are shadowed by "open search"

---
 .../gestures/GestureSettingsScreen.kt         | 21 +++++++++++--------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/app/ui/src/main/java/de/mm20/launcher2/ui/settings/gestures/GestureSettingsScreen.kt b/app/ui/src/main/java/de/mm20/launcher2/ui/settings/gestures/GestureSettingsScreen.kt
index 2f47bc5c..aed1d68b 100644
--- a/app/ui/src/main/java/de/mm20/launcher2/ui/settings/gestures/GestureSettingsScreen.kt
+++ b/app/ui/src/main/java/de/mm20/launcher2/ui/settings/gestures/GestureSettingsScreen.kt
@@ -70,7 +70,8 @@ fun GestureSettingsScreen() {
                     onValueChanged = { if (it != null) viewModel.setLongPress(it) }
                 )
                 val swipeDown by viewModel.swipeDown.observeAsState()
-                AnimatedVisibility(hasPermission == false && requiresAccessibilityService(swipeDown)) {
+                val swipeDownIsSearch = layout == Layout.PullDown
+                AnimatedVisibility(hasPermission == false && requiresAccessibilityService(swipeDown) && !swipeDownIsSearch) {
                     MissingPermissionBanner(
                         modifier = Modifier.padding(16.dp),
                         text = stringResource(R.string.missing_permission_accessibility_gesture_settings),
@@ -79,13 +80,14 @@ fun GestureSettingsScreen() {
                 }
                 ListPreference(
                     title = stringResource(R.string.preference_gesture_swipe_down),
-                    enabled = layout != Layout.PullDown,
+                    enabled = !swipeDownIsSearch,
                     items = options,
-                    value = if (layout == Layout.PullDown) GestureAction.OpenSearch else swipeDown,
+                    value = if (swipeDownIsSearch) GestureAction.OpenSearch else swipeDown,
                     onValueChanged = { if (it != null) viewModel.setSwipeDown(it) }
                 )
                 val swipeLeft by viewModel.swipeLeft.observeAsState()
-                AnimatedVisibility(hasPermission == false && requiresAccessibilityService(swipeLeft)) {
+                val swipeLeftIsSearch = layout == Layout.Pager
+                AnimatedVisibility(hasPermission == false && requiresAccessibilityService(swipeLeft) && !swipeLeftIsSearch) {
                     MissingPermissionBanner(
                         modifier = Modifier.padding(16.dp),
                         text = stringResource(R.string.missing_permission_accessibility_gesture_settings),
@@ -94,13 +96,14 @@ fun GestureSettingsScreen() {
                 }
                 ListPreference(
                     title = stringResource(R.string.preference_gesture_swipe_left),
-                    enabled = layout != Layout.Pager,
+                    enabled = !swipeLeftIsSearch,
                     items = options,
-                    value = if (layout == Layout.Pager) GestureAction.OpenSearch else swipeLeft,
+                    value = if (swipeLeftIsSearch) GestureAction.OpenSearch else swipeLeft,
                     onValueChanged = { if (it != null) viewModel.setSwipeLeft(it) }
                 )
                 val swipeRight by viewModel.swipeRight.observeAsState()
-                AnimatedVisibility(hasPermission == false && requiresAccessibilityService(swipeRight)) {
+                val swipeRightIsSearch = layout == Layout.PagerReversed
+                AnimatedVisibility(hasPermission == false && requiresAccessibilityService(swipeRight) && !swipeRightIsSearch) {
                     MissingPermissionBanner(
                         modifier = Modifier.padding(16.dp),
                         text = stringResource(R.string.missing_permission_accessibility_gesture_settings),
@@ -109,9 +112,9 @@ fun GestureSettingsScreen() {
                 }
                 ListPreference(
                     title = stringResource(R.string.preference_gesture_swipe_right),
-                    enabled = layout != Layout.PagerReversed,
+                    enabled = !swipeRightIsSearch,
                     items = options,
-                    value = if (layout == Layout.PagerReversed) GestureAction.OpenSearch else swipeRight,
+                    value = if (swipeRightIsSearch) GestureAction.OpenSearch else swipeRight,
                     onValueChanged = { if (it != null) viewModel.setSwipeRight(it) }
                 )
             }