refactor: remove steps in sliders in compose

anilbeesetti · Aug 19, 2023 · 4847d89 · 4847d89
1 parent a5d349b
commit 4847d89
Showing 1 changed file with 7 additions and 11 deletions.
diff --git a/.../main/java/dev/anilbeesetti/nextplayer/settings/screens/player/PlayerPreferencesScreen.kt b/.../main/java/dev/anilbeesetti/nextplayer/settings/screens/player/PlayerPreferencesScreen.kt
@@ -42,7 +42,6 @@ import dev.anilbeesetti.nextplayer.core.ui.designsystem.NextIcons
 import dev.anilbeesetti.nextplayer.settings.composables.OptionsDialog
 import dev.anilbeesetti.nextplayer.settings.composables.PreferenceSubtitle
 import dev.anilbeesetti.nextplayer.settings.extensions.name
-import java.lang.Exception
 import java.util.Locale
 
 @OptIn(ExperimentalMaterial3Api::class)
@@ -260,11 +259,8 @@ fun PlayerPreferencesScreen(
                         )
                         Slider(
                             value = defaultPlaybackSpeed,
-                            onValueChange = {
-                                defaultPlaybackSpeed = it.round(2)
-                            },
-                            valueRange = 0.2f..4.0f,
-                            steps = 37
+                            onValueChange = { defaultPlaybackSpeed = it.round(1) },
+                            valueRange = 0.2f..4.0f
                         )
                     }
                 )
@@ -294,8 +290,7 @@ fun PlayerPreferencesScreen(
                         Slider(
                             value = controllerAutoHideSec.toFloat(),
                             onValueChange = { controllerAutoHideSec = it.toInt() },
-                            valueRange = 1.0f..60.0f,
-                            steps = 60
+                            valueRange = 1.0f..60.0f
                         )
                     }
                 )
@@ -325,8 +320,7 @@ fun PlayerPreferencesScreen(
                         Slider(
                             value = seekIncrement.toFloat(),
                             onValueChange = { seekIncrement = it.toInt() },
-                            valueRange = 1.0f..60.0f,
-                            steps = 60
+                            valueRange = 1.0f..60.0f
                         )
                     }
                 )
@@ -438,7 +432,9 @@ fun LazyListScope.autoplaySetting(
         isChecked = isChecked,
         onClick = onClick
     )
-} fun LazyListScope.rememberBrightnessSetting(
+}
+
+fun LazyListScope.rememberBrightnessSetting(
     isChecked: Boolean,
     onClick: () -> Unit
 ) = item {