experiments/Experiment-compose-learning
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix order of operations in NumberPicker function

Browse source
This commit is contained in:
Gergely Hegedus 2022-06-14 17:34:09 +03:00
parent 0b4fa9ece7
commit 087bd4ea7e
2 changed files with 8 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

10
picker/src/main/java/org/fknives/android/compose/picker/number/NumberPicker.kt
View file

@ -4,8 +4,8 @@ import androidx.compose.material.LocalTextStyle
import androidx.compose.runtime.Composable
import androidx.compose.ui.Modifier
import androidx.compose.ui.text.TextStyle
import org.fknives.android.compose.picker.text.util.TextPickerDefaults
import org.fknives.android.compose.picker.text.TextPickerState
import org.fknives.android.compose.picker.text.util.TextPickerDefaults
@Composable
fun NumberPicker(
@ -43,10 +43,10 @@ fun NumberPicker(
require(selectedValue >= config.minimum) { "Selected Value($selectedValue) is less than Minimum (${config.minimum})!" }
require(selectedValue <= config.maximum) { "Selected Value($selectedValue) is more than Maximum (${config.maximum})!" }
val numberPickerScope = rememberNumberPickerScope(
state,
config,
onIndexDifferenceChanging,
onSelectedChange
state = state,
config = config,
onIndexDifferenceChanging = onIndexDifferenceChanging,
onSelectedChange = onSelectedChange
)
timePicker(numberPickerScope)

4
picker/src/main/java/org/fknives/android/compose/picker/number/NumberPickerScope.kt
View file

@ -33,7 +33,9 @@ fun rememberNumberPickerScope(
state = state,
textForIndex = { "${indexToNumber(it)}" },
onIndexDifferenceChanging = onIndexDifferenceChanging,
onSelectedIndexChange = { onSelectedChange(indexToNumber(it)) }
onSelectedIndexChange = {
onSelectedChange(indexToNumber(it))
}
)
}
}