PLASMA-8121: feat(sdds-acore/uikit-compose): Stateful value migration for Segment, ComboBox, Autocomplete - #911
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
📦 Собранные Release APK
📲 Как установить APK на Android-устройство:
💻 Установка через USB (ADB):
|
| @@ -63,16 +68,58 @@ fun ComboBox( | |||
| header: (@Composable () -> Unit)? = null, | |||
| footer: (@Composable () -> Unit)? = null, | |||
| dropdownProperties: DropdownProperties = ComboBoxDefaultDropdownProperties, | |||
There was a problem hiding this comment.
кажется, что можно было не добавлять новую перегрузку, а докинуть последний параметр в существующий.
There was a problem hiding this comment.
Они добавлены, потому что только так соблюдается бинарная совместимость. Если добавить параметр со значение по умолчанию, старая сигнатура функции будет потеряна. Для вновь компилируемого кода это не проблема, а вот скомпилированный код не найдёт эту функцию. Проверил в байткоде.
| visualTransformation: VisualTransformation = VisualTransformation.None, | ||
| focusSelectorSettings: FocusSelectorSettings = LocalFocusSelectorSettings.current, | ||
| interactionSource: MutableInteractionSource = remember { MutableInteractionSource() }, | ||
| ) { |
There was a problem hiding this comment.
и тут,
проверь плиз везде такие кейсы
sdds-uikit-compose
Segment
SegmentItem
Autocomplete
ComboBox
What/why changed
Компоненты Segment, SegmentItem, Autocomplete, Combobox переведены на StatefulValue.