FlowRow affects Modifier.height(IntrinsicSize)

[C5FR7Q]

Describe the bug

FlowRow somehow affects Modifier.height(IntrinsicSize.Max), that's so resulting height is wrong

To Reproduce

Steps to reproduce the behavior:
Use LazyColumn the items of which are some containers with Modifier.height(IntrinsicSize.Max) and including FlowRow with different items count.

Example:

@Composable
fun Screen() {
	val items = listOf<List<String>>(
		listOf("a", "b", "c"),
		listOf("a", "b", "c", "d"),
		listOf("a"),
		listOf("a", "b"),
		listOf("c")
	)
	LazyColumn {
		items(items) {
			Item(it)
		}
	}
}

@Composable
fun Item(strings: List<String>) {
	Box(
		modifier = Modifier
			.fillMaxWidth()
			.height(IntrinsicSize.Max)
	) {
		Spacer(
			modifier = Modifier
				.background(Color(Random.nextInt(), Random.nextInt(), Random.nextInt()))
				.width(4.dp)
				.fillMaxHeight()
				.align(Alignment.CenterStart)
		)
		FlowRow {
			strings.forEach {
				Text(text = it)
			}
		}
	}
}

Expected behavior

(In accordance with example code) Resulting height of Item is equal to the height of FlowRow

Actual behavior

Item's height somehow depends on FlowRow items count.

Environment:

• Android OS version: Android 10
• Device: Nokia 5.3
• Accompanist version: 0.16.0
• Compose version: 1.0.0

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[C5FR7Q]

Waiting for #650 to be merged

[mihirrai]

Facing a similar issue where FlowRow is somehow adding extra height to the layout.

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[PaulWoitaschek]

Remove the stale label please, this forced us to create our own flowlayout.

[6uilhermeAlmeida]

@C5FR7Q Any way you could re-open the issue? The PR that was supposed to fix it seems abandoned

[C5FR7Q]

Waiting for #650 to be merged or any other fix

[PaulWoitaschek]

@bentrengrove
Could you re-open this ticket please?

[bentrengrove]

We are currently working on an official version of Flow Layouts in Compose as seen on the Compose roadmap, as such I don't think we will get to fixing this bug in Accompanist. Flow Layouts in Accompanist will soon be deprecated with the release of Flow Layouts in Compose main.