Skip to content

Commit 9b2dbaa

Browse files
author
burak.ozturk1
committed
Add showLoading parameter to QuantityAddIcon for better loading state handling
1 parent 17d102d commit 9b2dbaa

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

libraries/quantity-picker-compose/src/main/java/com/trendyol/uicomponents/quantitypicker/QuantityAddIcon.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ import kotlinx.coroutines.launch
2626
internal fun QuantityAddIcon(
2727
icons: QuantityIcons,
2828
quantityData: QuantityPickerViewData,
29+
showLoading: Boolean,
2930
onAddClick: (() -> Unit)?
3031
) {
3132
val coroutineScope = rememberCoroutineScope()
@@ -78,7 +79,7 @@ internal fun QuantityAddIcon(
7879
interactionSource = MutableInteractionSource(),
7980
enabled = quantityData.isAddButtonEnabled(),
8081
onClick = {
81-
if (quantityData.currentQuantity == 0) {
82+
if (quantityData.currentQuantity == 0 && showLoading.not()) {
8283
setTargetBackgroundColor.invoke(Color.White)
8384
iconTintColor = icons.iconColor
8485
}

libraries/quantity-picker-compose/src/main/java/com/trendyol/uicomponents/quantitypicker/QuantityPicker.kt

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,8 @@ internal fun VerticalQuantityPicker(
8888
QuantityAddIcon(
8989
icons = icons,
9090
quantityData = quantityData,
91-
onAddClick = onAddClick
91+
onAddClick = onAddClick,
92+
showLoading = showLoading
9293
)
9394

9495
AnimatedVisibility(visible = quantityData.currentQuantity > 0 || showLoading) {
@@ -165,7 +166,8 @@ internal fun HorizontalQuantityPicker(
165166
QuantityAddIcon(
166167
icons = icons,
167168
quantityData = quantityData,
168-
onAddClick = onAddClick
169+
onAddClick = onAddClick,
170+
showLoading = showLoading
169171
)
170172
}
171173
}

0 commit comments

Comments
 (0)