Which is the most performing way to wrap a composa...

# compose

[manueldidonna]

Which is the most performing way to wrap a composable just to add a modifier?

[Se7eN]

I use a Box

[caelum19]

Can't pass a Modifier parameter?

[manueldidonna]

I can't pass a Modifier to achieve something like this:

MenuOptions(callbacks) {
    ComposableToWrap()
}

[manueldidonna]

I use Box too

[manueldidonna]

I've found something useful from compose source code ui/platform/Wrapper.kt

@Composable
private fun simpleLayout(modifier: Modifier = Modifier, children: @Composable () -> Unit) {
    Layout(modifier = modifier, children = children) { measurables, constraints ->
        val placeables = measurables.map { measurable ->
            measurable.measure(constraints)
        }

        val width = placeables.fold(0) { maxWidth, placeable ->
            max(maxWidth, (placeable.width))
        }

        val height = placeables.fold(0) { minWidth, placeable ->
            max(minWidth, (placeable.height))
        }

        layout(width, height) {
            placeables.forEach { placeable ->
                placeable.place(0, 0)
            }
        }
    }
}

[Se7eN]

I'd still prefer a Box though. https://foso.github.io/Jetpack-Compose-Playground/foundation/box/

[manueldidonna]

@Se7eN why?

[manueldidonna]

Anyway Box will be refactored soon. It will replace Stack

[Se7eN]

Damn I didn't know that. Check out the link it says a box can be used as a wrapper composable, but of course it's not the official docs.

[manueldidonna]

Yeah, I've seen these docs. Now Box under the hood is a Column with same sane defaults. I think we should analyze the performance impact of a box over simpleLayout() to just add a Modifier to a composabe. Maybe it's irrilevant.

[caelum19]

What about adding a Modifier as a parameter to the ComposableToWrap function? like children: @Composable (Modifier) -> Unit ?