Hey Why is it that there s no easy way to combine or modify

Question

Hey Why is it that there s no easy way to combine or modify `PaddingValues` Sometimes I would like to combine padding values like `WindowInsets ime asPaddingValues PaddingValues bottom = 30 dp ` Other times I would just like to override some default padding values like `BottomSheetDefaults ContentPadding copy bottom = 0 dp ` Shouldn t this be easier in the compose APIs Or am I missing the right way to do this

Zach Klippenstein (he/him) [MOD] · Answer

Please file a feature request

Quinten Schelfhout · Answer

I create an extension for + and on PaddingValues I also found it weird that this is not included

gsala · Answer

Filed at <https issuetracker google com issues 244468479>

gsala · Answer

Would this extension function make sense ``` Composable operator fun PaddingValues plus other PaddingValues PaddingValues return PaddingValues start = calculateStartPadding LayoutDirection Ltr + other calculateStartPadding LayoutDirection Ltr top = calculateTopPadding + other calculateTopPadding end = calculateEndPadding LayoutDirection Ltr + other calculateEndPadding LayoutDirection Ltr bottom = calculateBottomPadding + other calculateBottomPadding ```

Adam Powell · Answer

The hardcoded LTR is going to give you a bad time

Adam Powell · Answer

You re going to have to return a PaddingValues that holds onto both inputs so it can delegate with the right layout direction when asked

Zoltan Demant · Answer

Was just going to say that probably huge headache when RTL support lands in the future and the padding is off for some reason ~LocalLayoutDirection current~

Adam Powell · Answer

Nope not `LocalLayoutDirection` either

Adam Powell · Answer

Calculate on the fly

Zoltan Demant · Answer

Oh I think I understand what you mean

Adam Powell · Answer

Bidi padding when mixed relative and absolute paddings are involved is a nightmare

Adam Powell · Answer

Anything you try to cache or precompute will be a bug later

gsala · Answer

So there s no straightforward solution for this I d have to make a new implementation of `PaddingValues` The issue I filed has people mentioning that it shouldn t be a problem to support this I was hoping to just have it now instead of waiting for it to be implemented in foundation and released

Zoltan Demant · Answer

Couldnt you do something like this ```class CombinedPaddingValues val first PaddingValues val second PaddingValues PaddingValues override fun calculateLeftPadding layoutDirection LayoutDirection Dp return first calculateLeftPadding layoutDirection + second calculateLeftPadding layoutDirection ```

gsala · Answer

What if I want to implement a minus then Another implementation

Zoltan Demant · Answer

Its far from ideal especially if you need to do it yourself as compared to it already being in one of the compose libraries Ive starred the issue