Skip to content

Commit 4172843

Browse files
authored
Merge pull request #14935 from woocommerce/fix/pos-historical-orders-long-product-names
Fix overlapping texts by pinning to the text bottom
2 parents 8834f98 + 65318c0 commit 4172843

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/woopos/orders/WooPosOrdersDetails.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ private fun OrderProductItem(row: OrderDetailsViewState.Computed.Details.LineIte
170170
style = WooPosTypography.BodyMedium,
171171
color = WooPosTheme.colors.onSurfaceVariantHighest,
172172
modifier = Modifier.constrainAs(qtyText) {
173-
bottom.linkTo(image.bottom)
173+
top.linkTo(nameText.bottom, margin = WooPosSpacing.XSmall.value)
174174
start.linkTo(nameText.start)
175175
end.linkTo(totalText.start, margin = WooPosSpacing.Small.value)
176176
width = Dimension.fillToConstraints
@@ -357,7 +357,7 @@ fun WooPosOrderDetailsPreview() {
357357
lineItems = listOf(
358358
OrderDetailsViewState.Computed.Details.LineItemRow(101, "Cup", "2 x $4.00", "$8.00", null),
359359
OrderDetailsViewState.Computed.Details.LineItemRow(102, "Coffee Container", "1 x $10.00", "$10.00", null),
360-
OrderDetailsViewState.Computed.Details.LineItemRow(103, "Paper Filter", "1 x $5.00", "$5.00", null)
360+
OrderDetailsViewState.Computed.Details.LineItemRow(103, "A vey tasty coffee that incidentally has a very long name and should go over a few lines without overlapping anything", "1 x $5.00", "$5.00", null)
361361
),
362362
breakdown = OrderDetailsViewState.Computed.Details.TotalsBreakdown(
363363
products = "$23.00",

0 commit comments

Comments
 (0)