From cf30088c7ef39d74c486b9e57ceb62e355187194 Mon Sep 17 00:00:00 2001
From: starry-shivam <krshivam24@proton.me>
Date: Tue, 17 Oct 2023 16:42:59 +0530
Subject: [PATCH] Fix flashing of material you warning message on devices below
 A12

Signed-off-by: starry-shivam <krshivam24@proton.me>
---
 .idea/deploymentTargetDropDown.xml            |  4 +--
 .idea/misc.xml                                |  9 +-----
 .../settings/composables/SettingsItems.kt     |  8 +++--
 .../settings/composables/SettingsScreen.kt    | 30 +++++++++----------
 4 files changed, 24 insertions(+), 27 deletions(-)

diff --git a/.idea/deploymentTargetDropDown.xml b/.idea/deploymentTargetDropDown.xml
index 7f9a9933..c5b1756a 100644
--- a/.idea/deploymentTargetDropDown.xml
+++ b/.idea/deploymentTargetDropDown.xml
@@ -7,11 +7,11 @@
         <deviceKey>
           <Key>
             <type value="VIRTUAL_DEVICE_PATH" />
-            <value value="C:\Users\starr\.android\avd\Pixel_6_Pro_API_26.avd" />
+            <value value="C:\Users\starr\.android\avd\Pixel_4a_API_33.avd" />
           </Key>
         </deviceKey>
       </Target>
     </targetSelectedWithDropDown>
-    <timeTargetWasSelectedWithDropDown value="2023-09-01T06:11:02.691526400Z" />
+    <timeTargetWasSelectedWithDropDown value="2023-10-17T11:09:25.944640600Z" />
   </component>
 </project>
\ No newline at end of file
diff --git a/.idea/misc.xml b/.idea/misc.xml
index 9f94ed30..8978d23d 100644
--- a/.idea/misc.xml
+++ b/.idea/misc.xml
@@ -1,16 +1,9 @@
 <project version="4">
   <component name="ExternalStorageConfigurationManager" enabled="true" />
-  <component name="ProjectRootManager" version="2" languageLevel="JDK_17" project-jdk-name="jbr-17" project-jdk-type="JavaSDK">
+  <component name="ProjectRootManager" version="2" languageLevel="JDK_17" default="true" project-jdk-name="jbr-17" project-jdk-type="JavaSDK">
     <output url="file://$PROJECT_DIR$/build/classes" />
   </component>
   <component name="ProjectType">
     <option name="id" value="Android" />
   </component>
-  <component name="SuppressABINotification">
-    <option name="modulesWithSuppressedNotConfigured">
-      <set>
-        <option value="Myne.app" />
-      </set>
-    </option>
-  </component>
 </project>
\ No newline at end of file
diff --git a/app/src/main/java/com/starry/myne/ui/screens/settings/composables/SettingsItems.kt b/app/src/main/java/com/starry/myne/ui/screens/settings/composables/SettingsItems.kt
index c9ec6331..812a94c5 100644
--- a/app/src/main/java/com/starry/myne/ui/screens/settings/composables/SettingsItems.kt
+++ b/app/src/main/java/com/starry/myne/ui/screens/settings/composables/SettingsItems.kt
@@ -127,7 +127,11 @@ fun SettingItem(icon: Int, mainText: String, subText: String, onClick: () -> Uni
 @ExperimentalMaterial3Api
 @Composable
 fun SettingItemWIthSwitch(
-    icon: Int, mainText: String, subText: String, switchState: MutableState<Boolean>
+    icon: Int,
+    mainText: String,
+    subText: String,
+    switchState: MutableState<Boolean>,
+    onCheckChange: (Boolean) -> Unit
 ) {
     Card(
         colors = CardDefaults.cardColors(
@@ -186,7 +190,7 @@ fun SettingItemWIthSwitch(
                     )
                 }
             }
-            Switch(checked = switchState.value, onCheckedChange = { switchState.value = it })
+            Switch(checked = switchState.value, onCheckedChange = onCheckChange)
         }
     }
 }
\ No newline at end of file
diff --git a/app/src/main/java/com/starry/myne/ui/screens/settings/composables/SettingsScreen.kt b/app/src/main/java/com/starry/myne/ui/screens/settings/composables/SettingsScreen.kt
index f37944b7..c9fba15e 100644
--- a/app/src/main/java/com/starry/myne/ui/screens/settings/composables/SettingsScreen.kt
+++ b/app/src/main/java/com/starry/myne/ui/screens/settings/composables/SettingsScreen.kt
@@ -50,12 +50,13 @@ import coil.annotation.ExperimentalCoilApi
 import com.starry.myne.BuildConfig
 import com.starry.myne.MainActivity
 import com.starry.myne.R
-import com.starry.myne.ui.navigation.Screens
 import com.starry.myne.ui.common.CustomTopAppBar
+import com.starry.myne.ui.navigation.Screens
 import com.starry.myne.ui.screens.settings.viewmodels.SettingsViewModel
 import com.starry.myne.ui.screens.settings.viewmodels.ThemeMode
 import com.starry.myne.ui.theme.figeronaFont
 import com.starry.myne.utils.getActivity
+import kotlinx.coroutines.launch
 
 @SuppressLint("UnusedMaterial3ScaffoldPaddingParameter")
 @ExperimentalCoilApi
@@ -274,6 +275,8 @@ fun DisplayOptionsUI(
     context: Context,
     snackbarHostState: SnackbarHostState,
 ) {
+    val coroutiScope = rememberCoroutineScope()
+
     val displayValue =
         when (viewModel.getThemeValue()) {
             ThemeMode.Light.ordinal -> "Light"
@@ -312,23 +315,20 @@ fun DisplayOptionsUI(
             icon = R.drawable.ic_settings_material_you,
             mainText = stringResource(id = R.string.material_you_setting),
             subText = materialYouDesc,
-            switchState = materialYouSwitch
+            switchState = materialYouSwitch,
+            onCheckChange = { materialYouValue ->
+                if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
+                    viewModel.setMaterialYou(materialYouValue)
+                    materialYouSwitch.value = materialYouValue
+                } else {
+                    viewModel.setMaterialYou(false)
+                    materialYouSwitch.value = false
+                    coroutiScope.launch { snackbarHostState.showSnackbar(context.getString(R.string.material_you_error)) }
+                }
+            }
         )
     }
 
-    if (materialYouSwitch.value) {
-        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
-            viewModel.setMaterialYou(true)
-        } else {
-            materialYouSwitch.value = false
-            LaunchedEffect(
-                key1 = true,
-                block = { snackbarHostState.showSnackbar(context.getString(R.string.material_you_error)) })
-        }
-    } else {
-        viewModel.setMaterialYou(false)
-    }
-
     if (displayDialog.value) {
         AlertDialog(onDismissRequest = {
             displayDialog.value = false