Fix a layout bug in Yoga (#2347)

* Fix a layout bug in Yoga Keep independent variables for the childWidthRef and childHeightRef. * Fix changelog
cashapp · Oct 1, 2024 · 7db5420 · 7db5420
1 parent 98c04fc
commit 7db5420
Show file tree

Hide file tree

Showing 7 changed files with 15 additions and 13 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,7 +10,7 @@ Changed:
 - Nothing yet!
 
 Fixed:
-- Nothing yet!
+- Fix a layout bug where children of fixed-with `Row` containers were assigned the wrong width.
 
 
 ## [0.15.0] - 2024-09-30

diff --git a/...omposeUiFlexContainerTest_testRowWithFixedWidthHasChildWithFixedHeight[LTR].png b/...omposeUiFlexContainerTest_testRowWithFixedWidthHasChildWithFixedHeight[LTR].png
diff --git a/...omposeUiFlexContainerTest_testRowWithFixedWidthHasChildWithFixedHeight[RTL].png b/...omposeUiFlexContainerTest_testRowWithFixedWidthHasChildWithFixedHeight[RTL].png
diff --git a/.../UIViewFlexContainerTestHost/testRowWithFixedWidthHasChildWithFixedHeight.1.png b/.../UIViewFlexContainerTestHost/testRowWithFixedWidthHasChildWithFixedHeight.1.png
diff --git a/...iew_ViewFlexContainerTest_testRowWithFixedWidthHasChildWithFixedHeight[LTR].png b/...iew_ViewFlexContainerTest_testRowWithFixedWidthHasChildWithFixedHeight[LTR].png
diff --git a/...iew_ViewFlexContainerTest_testRowWithFixedWidthHasChildWithFixedHeight[RTL].png b/...iew_ViewFlexContainerTest_testRowWithFixedWidthHasChildWithFixedHeight[RTL].png
diff --git a/redwood-yoga/src/commonMain/kotlin/app/cash/redwood/yoga/internal/Yoga.kt b/redwood-yoga/src/commonMain/kotlin/app/cash/redwood/yoga/internal/Yoga.kt
@@ -1745,15 +1745,17 @@ internal object Yoga {
         mode = childWidthMeasureMode,
         size = childWidthRef,
       )
+      childWidth = childWidthRef.argValue
+      val childHeightRef = RefObject(childHeight)
       YGConstrainMaxSizeForMode(
         node = child,
         axis = YGFlexDirection.YGFlexDirectionColumn,
         ownerAxisSize = ownerHeight,
         ownerWidth = ownerWidth,
         mode = childHeightMeasureMode,
-        size = childWidthRef,
+        size = childHeightRef,
       )
-      childWidth = childWidthRef.argValue
+      childHeight = childHeightRef.argValue
       YGLayoutNodeInternal(
         node = child,
         availableWidth = childWidth,