Skip to content

Commit

Permalink
Fix cross sell placeholder shapes
Browse files Browse the repository at this point in the history
The top text needs a smaller corner size since it is a smaller box.
  • Loading branch information
StylianosGakis committed Nov 15, 2024
1 parent 9e42e6a commit 5ebd3fa
Showing 1 changed file with 7 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -121,11 +121,7 @@ private fun CrossSellItem(
contentDescription = null,
modifier = Modifier
.size(48.dp)
.hedvigPlaceholder(
visible = isLoading,
highlight = PlaceholderHighlight.fade(),
shape = HedvigTheme.shapes.cornerLarge,
),
.hedvigPlaceholder(visible = isLoading, highlight = PlaceholderHighlight.fade()),
)
Spacer(Modifier.width(16.dp))
Column(
Expand All @@ -135,7 +131,11 @@ private fun CrossSellItem(
HedvigText(
text = crossSellTitle,
style = HedvigTheme.typography.bodySmall,
modifier = Modifier.hedvigPlaceholder(visible = isLoading, highlight = PlaceholderHighlight.shimmer()),
modifier = Modifier.hedvigPlaceholder(
visible = isLoading,
highlight = PlaceholderHighlight.shimmer(),
shape = HedvigTheme.shapes.cornerXSmall,
),
)
Spacer(Modifier.height(4.dp))
HedvigText(
Expand All @@ -153,11 +153,7 @@ private fun CrossSellItem(
},
buttonSize = Small,
buttonStyle = PrimaryAlt,
modifier = Modifier.hedvigPlaceholder(
visible = isLoading,
highlight = PlaceholderHighlight.shimmer(),
shape = HedvigTheme.shapes.cornerXLarge,
),
modifier = Modifier.hedvigPlaceholder(visible = isLoading, highlight = PlaceholderHighlight.shimmer()),
enabled = !isLoading,
)
}
Expand Down

0 comments on commit 5ebd3fa

Please sign in to comment.