Remove baseline weirdness

4 files changed, 23 insertions, 37 deletions

diff --git a/src/library/flow.rs b/src/library/flow.rs
index 9494d6c0..98b518b7 100644
--- a/src/library/flow.rs
+++ b/src/library/flow.rs
@@ -214,38 +214,28 @@ impl<'a> FlowLayouter<'a> {
         }
 
         let mut output = Frame::new(size);
-        let mut before = Length::zero();
+        let mut offset = Length::zero();
         let mut ruler = Align::Top;
-        let mut first = true;
 
         // Place all frames.
         for item in self.items.drain(..) {
             match item {
                 FlowItem::Absolute(v) => {
-                    before += v;
+                    offset += v;
                 }
                 FlowItem::Fractional(v) => {
-                    before += v.resolve(self.fr, remaining);
+                    offset += v.resolve(self.fr, remaining);
                 }
                 FlowItem::Frame(frame, aligns) => {
                     ruler = ruler.max(aligns.y);
-
-                    // Align horizontally and vertically.
                     let x = aligns.x.resolve(size.x - frame.size.x);
-                    let y = before + ruler.resolve(size.y - self.used.y);
+                    let y = offset + ruler.resolve(size.y - self.used.y);
                     let pos = Point::new(x, y);
-                    before += frame.size.y;
-
-                    // The baseline of the flow is that of the first frame.
-                    if first {
-                        output.baseline = pos.y + frame.baseline;
-                        first = false;
-                    }
-
+                    offset += frame.size.y;
                     output.push_frame(pos, frame);
                 }
                 FlowItem::Placed(frame) => {
-                    output.push_frame(Point::with_y(before), frame);
+                    output.push_frame(Point::with_y(offset), frame);
                 }
             }
         }
diff --git a/src/library/pad.rs b/src/library/pad.rs
index 05cc30b7..681da73e 100644
--- a/src/library/pad.rs
+++ b/src/library/pad.rs
@@ -51,7 +51,6 @@ impl Layout for PadNode {
             // Grow the frame and translate everything in the frame inwards.
             let frame = Rc::make_mut(frame);
             frame.size = padded;
-            frame.baseline += offset.y;
             frame.translate(offset);
 
             // Set exact and base constraints if the child had them. Also set
diff --git a/src/library/par.rs b/src/library/par.rs
index c29e5f9e..6abfa7af 100644
--- a/src/library/par.rs
+++ b/src/library/par.rs
@@ -466,14 +466,18 @@ impl<'a> LineLayout<'a> {
 
         // Measure the size of the line.
         for item in first.iter().chain(items).chain(&last) {
-            match *item {
-                ParItem::Absolute(v) => width += v,
-                ParItem::Fractional(v) => fr += v,
-                ParItem::Text(ShapedText { size, baseline, .. })
-                | ParItem::Frame(Frame { size, baseline, .. }) => {
-                    width += size.x;
-                    top.set_max(baseline);
-                    bottom.set_max(size.y - baseline);
+            match item {
+                ParItem::Absolute(v) => width += *v,
+                ParItem::Fractional(v) => fr += *v,
+                ParItem::Text(shaped) => {
+                    width += shaped.size.x;
+                    top.set_max(shaped.baseline);
+                    bottom.set_max(shaped.size.y - shaped.baseline);
+                }
+                ParItem::Frame(frame) => {
+                    width += frame.size.x;
+                    top.set_max(frame.baseline());
+                    bottom.set_max(frame.size.y - frame.baseline());
                 }
             }
         }
@@ -496,9 +500,9 @@ impl<'a> LineLayout<'a> {
         let size = Size::new(self.size.x.max(width), self.size.y);
         let remaining = size.x - self.size.x;
 
-        let mut output = Frame::new(size);
         let mut offset = Length::zero();
-        output.baseline = self.baseline;
+        let mut output = Frame::new(size);
+        output.baseline = Some(self.baseline);
 
         for (range, item) in self.reordered() {
             let mut position = |mut frame: Frame| {
@@ -510,7 +514,7 @@ impl<'a> LineLayout<'a> {
                 }
 
                 let x = offset + self.par.align.resolve(remaining);
-                let y = self.baseline - frame.baseline;
+                let y = self.baseline - frame.baseline();
                 offset += frame.size.x;
 
                 // Add to the line's frame.
@@ -633,17 +637,10 @@ impl<'a> LineStack<'a> {
 
         let mut output = Frame::new(self.size);
         let mut offset = Length::zero();
-        let mut first = true;
 
         for line in self.lines.drain(..) {
             let frame = line.build(ctx, self.size.x);
-
             let pos = Point::with_y(offset);
-            if first {
-                output.baseline = pos.y + frame.baseline;
-                first = false;
-            }
-
             offset += frame.size.y + self.leading;
             output.merge_frame(pos, frame);
         }
diff --git a/src/library/text.rs b/src/library/text.rs
index aac99e8a..0790196d 100644
--- a/src/library/text.rs
+++ b/src/library/text.rs
@@ -305,9 +305,9 @@ pub struct ShapedGlyph {
 impl<'a> ShapedText<'a> {
     /// Build the shaped text's frame.
     pub fn build(&self) -> Frame {
-        let mut frame = Frame::new(self.size);
         let mut offset = Length::zero();
-        frame.baseline = self.baseline;
+        let mut frame = Frame::new(self.size);
+        frame.baseline = Some(self.baseline);
 
         for (face_id, group) in self.glyphs.as_ref().group_by_key(|g| g.face_id) {
             let pos = Point::new(offset, self.baseline);