Padding with negative values used to work before now it crashes runtime no compilation warnings given ```LayoutPadding bottom = 10 dp end = 5 dp ``` Beyond the crash what s the suggested way to achieve the same positioning

Padding with negative values used to work before now it cras kotlinlang #compose

Channels

portugal

romania

stayhungrystayfoolish

skia-wasm-interop-temp

java-to-kotlin-refactoring

event21-community-content

refactoring-to-kotlin

kotlin-multiplatform-contest

tricity-kotlin-user-group

cincinnati-user-group

prozis-android-backup

russian-kotlinasfirst

juul-libraries

lanark

wimix_sentry

androidgithubprojects

swagger-gradle-codegen

competitive-programming

navigation-architecture-component

myndocs-oauth2-server

practical-functional-programming

hyderabad

kotlinultimatechallenge

jvm-ir-backend-feedback

Title

# compose

Zsolt

03/19/2020, 11:03 PM

Padding with negative values used to work before, now it crashes runtime (no compilation warnings given):

Copy code

LayoutPadding(bottom = (-10).dp, end = (-5).dp)

Beyond the crash, what's the suggested way to achieve the same positioning?

java.lang.IllegalArgumentException: Padding must be non-negative

Leland Richardson [G]

03/19/2020, 11:37 PM

@Mihai Popa what’s our recommendation here? depends on the use case??? @Zsolt what is it you’re trying to do? Are you trying to “undo” another padding, or are you trying to actually do like a negative offset? I think we have an Offset modifier now which will allow negative values

Zsolt

03/20/2020, 2:14 AM

No undoing, just positioning background decorations so that they're slightly off the screen. LayoutOffset works fine, thanks!

Minor ux complaint: it doesn't have default values of 0.dp, so you always need to specify both x and y values.

Leland Richardson [G]

03/20/2020, 2:49 AM

yeah, good point

Mihai Popa

03/20/2020, 2:42 PM

Yes,

LayoutOffset

is the correct replacement for any padding that is only intended to offset the position

Thanks for the feedback regarding the API, we had deliberately decided to not put defaults, but your input is useful as we might decide to change it in the future 🙂

34 Views