From 79638d4bbdc140a8dca0ccfdf70ffd607d5da251 Mon Sep 17 00:00:00 2001
From: Laurenz <laurmaedje@gmail.com>
Date: Tue, 16 Nov 2021 12:47:14 +0100
Subject: Don't wrap already layoutable nodes into flows/pars unnecessarily

---
 src/eval/template.rs | 15 +++++++++------
 src/eval/walk.rs     |  6 +++---
 2 files changed, 12 insertions(+), 9 deletions(-)

(limited to 'src/eval')

diff --git a/src/eval/template.rs b/src/eval/template.rs
index 82a069f9..0604cc05 100644
--- a/src/eval/template.rs
+++ b/src/eval/template.rs
@@ -149,12 +149,15 @@ impl Template {
         Self(Rc::new(vec![TemplateNode::Decorated(deco, self)]))
     }
 
-    /// Build the flow node resulting from instantiating the template with the
-    /// given style.
-    pub fn to_flow(&self, style: &Style) -> FlowNode {
-        let mut builder = Builder::new(style, false);
-        builder.template(self);
-        builder.build_flow()
+    /// Pack the template into a layout node.
+    pub fn pack(&self, style: &Style) -> PackedNode {
+        if let [TemplateNode::Block(f) | TemplateNode::Inline(f)] = self.0.as_slice() {
+            f(style)
+        } else {
+            let mut builder = Builder::new(style, false);
+            builder.template(self);
+            builder.build_flow().pack()
+        }
     }
 
     /// Build the layout tree resulting from instantiating the template with the
diff --git a/src/eval/walk.rs b/src/eval/walk.rs
index 134f10c7..fe7f0e98 100644
--- a/src/eval/walk.rs
+++ b/src/eval/walk.rs
@@ -124,7 +124,7 @@ impl Walk for EnumNode {
 
 fn walk_item(ctx: &mut EvalContext, label: EcoString, body: Template) {
     ctx.template += Template::from_block(move |style| {
-        let label = ParNode {
+        let label = Layout::pack(ParNode {
             dir: style.par.dir,
             leading: style.leading(),
             children: vec![ParChild::Text(
@@ -132,13 +132,13 @@ fn walk_item(ctx: &mut EvalContext, label: EcoString, body: Template) {
                 style.aligns.inline,
                 Rc::clone(&style.text),
             )],
-        };
+        });
 
         let spacing = style.text.size / 2.0;
         GridNode {
             tracks: Spec::new(vec![TrackSizing::Auto; 2], vec![]),
             gutter: Spec::new(vec![TrackSizing::Linear(spacing.into())], vec![]),
-            children: vec![label.pack(), body.to_flow(style).pack()],
+            children: vec![label, body.pack(style)],
         }
     });
 }
-- 
cgit v1.2.3