Fixed
Status Update
Comments
so...@google.com <so...@google.com> #2
This is a known issue with the Ripple. Assigning to Andrey.
an...@google.com <an...@google.com> #3
Yes, current workaround is to wrap Row into another layout - Box, for example. The long term solution will be Ripple reimplemented as a Modifier when it will be possible
vi...@gmail.com <vi...@gmail.com> #4
Okay, yeah that's the current workaround.
Description
Version of Gradle Plugin: 4.0.0-alpha09
Version of Gradle: 6.1
Version of Java: 1.8
OS: Android
Steps to Reproduce:
1. Update compose to dev-05 version
2. Padding component is missing, in favor of LayoutPadding
3. Replace Padding with LayoutPadding
Previous Code:
Ripple(bounded = true) {
Padding(padding = dimensionResource(id = R.dimen.padding)) {
Row {
restaurantImage(url = item.imageUrl)
Column(
modifier = LayoutFlexible(1f)
.plus(
LayoutPadding(
left = dimensionResource(id = R.dimen.padding),
right = dimensionResource(id = R.dimen.padding)
)
).plus(
LayoutGravity.Center
)
) {
restaurantInfo(item = item)
}
}
}
}
New Code:
Replace Padding with LayoutPadding as a modifier in Row.
P.S: Notice the ripple height and width in first vs second. First one takes draws ripple with padding into consideration and the second one doesn't. Though layout drawing similarly.