Change grid base for auto columns

library/src/layout/grid.rs

@@ -410,12 +410,10 @@ impl<'a, 'v> GridLayouter<'a, 'v> {
             for y in 0..self.rows.len() {
                 if let Some(cell) = self.cell(x, y) {
                     let size = Size::new(available, self.regions.base.y);
-                    let mut pod =
+                    let mut pod = Regions::one(size, size, Axes::splat(false));
-                        Regions::one(size, self.regions.base, Axes::splat(false));
 
                     // For relative rows, we can already resolve the correct
-                    // base, for auto it's already correct and for fr we could
+                    // base and for auto and fr we could only guess anyway.
-                    // only guess anyway.
                     if let TrackSizing::Relative(v) = self.rows[y] {
                         pod.base.y =
                             v.resolve(self.styles).relative_to(self.regions.base.y);
@@ -488,15 +486,10 @@ impl<'a, 'v> GridLayouter<'a, 'v> {
         // Determine the size for each region of the row.
         for (x, &rcol) in self.rcols.iter().enumerate() {
             if let Some(cell) = self.cell(x, y) {
-                let mut pod = self.regions.clone();
+                let mut pod = self.regions;
                 pod.first.x = rcol;
                 pod.base.x = rcol;
 
-                // All widths should be `rcol` except the base for auto columns.
-                if self.cols[x] == TrackSizing::Auto {
-                    pod.base.x = self.regions.base.x;
-                }
-
                 let mut sizes = cell
                     .layout(self.vt, self.styles, pod)?
                     .into_iter()
@@ -578,13 +571,13 @@ impl<'a, 'v> GridLayouter<'a, 'v> {
         for (x, &rcol) in self.rcols.iter().enumerate() {
             if let Some(cell) = self.cell(x, y) {
                 let size = Size::new(rcol, height);
-
+                let base = Size::new(
-                // Set the base to the region's base for auto rows and to the
+                    rcol,
-                // size for relative and fractional rows.
+                    match self.rows[y] {
-                let base = Axes::new(self.cols[x], self.rows[y])
+                        TrackSizing::Auto => self.regions.base.y,
-                    .map(|s| s == TrackSizing::Auto)
+                        _ => height,
-                    .select(self.regions.base, size);
+                    },
-
+                );
                 let pod = Regions::one(size, base, Axes::splat(true));
                 let frame = cell.layout(self.vt, self.styles, pod)?.into_frame();
                 output.push_frame(pos, frame);
@@ -616,11 +609,6 @@ impl<'a, 'v> GridLayouter<'a, 'v> {
                 pod.first.x = rcol;
                 pod.base.x = rcol;
 
-                // All widths should be `rcol` except the base for auto columns.
-                if self.cols[x] == TrackSizing::Auto {
-                    pod.base.x = self.regions.base.x;
-                }
-
                 // Push the layouted frames into the individual output frames.
                 let fragment = cell.layout(self.vt, self.styles, pod)?;
                 for (output, frame) in outputs.iter_mut().zip(fragment) {

tests/typ/bugs/grid-1.typ

@@ -0,0 +1,16 @@
+// Test that grid base for auto rows makes sense.
+
+---
+#set page(height: 150pt)
+#table(
+  columns: (1.5cm, auto),
+  rows: (auto, auto),
+  rect(width: 100%, fill: red),
+  rect(width: 100%, fill: blue),
+  rect(width: 100%, height: 50%, fill: green),
+)
+
+---
+#rect(width: 100%, height: 1em)
+- #rect(width: 100%, height: 1em)
+  - #rect(width: 100%, height: 1em)