Skip to content
This repository was archived by the owner on Dec 27, 2024. It is now read-only.

Make ConstraintLayout an inline composable #193

Merged
merged 1 commit into from
Mar 24, 2021
Merged

Conversation

mihaipopa12
Copy link
Contributor

This is consistent with how most layouts in foundation-layout are
implemented. In addition, it implicitly fixes issue 181717954, which was
happening because the content lambda could previously be recomposed
without recomposing the parent content lambda given to
MultiMeasureLayout - therefore, the scope was not reset() but it was
added the barrier again causing unexpected behavior.

Fixes: 181717954
Test: ConstraintLayoutTest

This is consistent with how most layouts in foundation-layout are
implemented. In addition, it implicitly fixes issue 181717954, which was
happening because the content lambda could previously be recomposed
without recomposing the parent content lambda given to
MultiMeasureLayout - therefore, the scope was not reset() but it was
added the barrier again causing unexpected behavior.

Fixes: 181717954
Test: ConstraintLayoutTest
@mihaipopa12 mihaipopa12 requested a review from camaelon March 24, 2021 15:08
@camaelon camaelon merged commit d89c45d into main Mar 24, 2021
@camaelon camaelon deleted the fix_barrier_crash branch March 24, 2021 15:53
@hrach
Copy link

hrach commented Mar 24, 2021

Wow, many thanks for fixing 181717954.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants