Skip to content

Commit 4634a7f

Browse files
authored
Merge pull request #138 from Trendyol/bugfix/quantityPickerComposeAddLoadingControl
Bugfix/quantity picker compose removed control
2 parents 4b1e4af + 66e7923 commit 4634a7f

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

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

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,6 @@ internal fun QuantityAddIcon(
7070
interactionSource = MutableInteractionSource(),
7171
enabled = quantityData.isAddButtonEnabled(),
7272
onClick = {
73-
if (quantityData.currentQuantity == 0) {
74-
setTargetBackgroundColor.invoke(Color.White)
75-
iconTintColor = icons.iconColor
76-
}
7773
onAddClick?.invoke()
7874
}
7975
)) {

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ internal fun VerticalQuantityPicker(
8989
QuantityAddIcon(
9090
icons = icons,
9191
quantityData = quantityData,
92-
onAddClick = onAddClick
92+
onAddClick = onAddClick,
9393
)
9494

9595
AnimatedVisibility(visible = quantityData.currentQuantity > 0 || showLoading) {
@@ -165,7 +165,7 @@ internal fun HorizontalQuantityPicker(
165165
QuantityAddIcon(
166166
icons = icons,
167167
quantityData = quantityData,
168-
onAddClick = onAddClick
168+
onAddClick = onAddClick,
169169
)
170170
}
171171
}

0 commit comments

Comments
 (0)