Simplify determineItemViewMode()

TeamNewPipe · Aug 10, 2024 · 75e18d1 · 75e18d1
1 parent d10c473
commit 75e18d1
Showing 1 changed file with 7 additions and 11 deletions.
diff --git a/app/src/main/java/org/schabi/newpipe/ui/components/items/ItemList.kt b/app/src/main/java/org/schabi/newpipe/ui/components/items/ItemList.kt
@@ -95,24 +95,20 @@ fun ItemList(
 
 @Composable
 private fun determineItemViewMode(): ItemViewMode {
-    val listMode = PreferenceManager.getDefaultSharedPreferences(LocalContext.current)
-        .getString(
-            stringResource(R.string.list_view_mode_key),
-            stringResource(R.string.list_view_mode_value)
-        )
+    val prefValue = PreferenceManager.getDefaultSharedPreferences(LocalContext.current)
+        .getString(stringResource(R.string.list_view_mode_key), null)
+    val viewMode = prefValue?.let { ItemViewMode.valueOf(it.uppercase()) } ?: ItemViewMode.AUTO
 
-    return when (listMode) {
-        stringResource(R.string.list_view_mode_list_key) -> ItemViewMode.LIST
-        stringResource(R.string.list_view_mode_grid_key) -> ItemViewMode.GRID
-        stringResource(R.string.list_view_mode_card_key) -> ItemViewMode.CARD
-        else -> {
-            // Auto mode - evaluate whether to use Grid based on screen real estate.
+    return when (viewMode) {
+        ItemViewMode.AUTO -> {
+            // Evaluate whether to use Grid based on screen real estate.
             val windowSizeClass = currentWindowAdaptiveInfo().windowSizeClass
             if (windowSizeClass.windowWidthSizeClass == WindowWidthSizeClass.EXPANDED) {
                 ItemViewMode.GRID
             } else {
                 ItemViewMode.LIST
             }
         }
+        else -> viewMode
     }
 }