From 543dc2ac078370d2bfab8312f9dc3b818c7a2e52 Mon Sep 17 00:00:00 2001
From: Peter Williams <peter@newton.cx>
Date: Sun, 11 Jun 2023 18:31:06 -0400
Subject: [PATCH 1/2] engine_spx2html: close automatic tags when starting
 certain automatic tags

---
 crates/engine_spx2html/src/emission.rs | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/crates/engine_spx2html/src/emission.rs b/crates/engine_spx2html/src/emission.rs
index 19ff5075b..c773755ae 100644
--- a/crates/engine_spx2html/src/emission.rs
+++ b/crates/engine_spx2html/src/emission.rs
@@ -424,6 +424,8 @@ impl EmittingState {
         match special {
             Special::AutoStartParagraph => {
                 if self.cur_elstate().do_auto_tags {
+                    self.close_automatics();
+
                     // Why are we using <div>s instead of <p>? As the HTML spec
                     // emphasizes, <p> tags are structural, not semantic. You cannot
                     // put tags like <ul> or <div> inside <p> -- they automatically
@@ -452,6 +454,7 @@ impl EmittingState {
                 if let Some(canvas) = self.current_canvas.as_mut() {
                     canvas.depth += 1;
                 } else {
+                    self.close_automatics();
                     self.current_canvas = Some(CanvasState::new(kind, x, y));
                 }
                 Ok(())
@@ -474,6 +477,7 @@ impl EmittingState {
             }
 
             Special::ManualFlexibleStart(spec) => {
+                self.close_automatics();
                 self.handle_flexible_start_tag(x, y, spec, common)
             }
 

From 58c5a0a3db76229e358fb45a0b75ff9dce950808 Mon Sep 17 00:00:00 2001
From: Peter Williams <peter@newton.cx>
Date: Sun, 11 Jun 2023 18:31:22 -0400
Subject: [PATCH 2/2] engine_spx2html: avoid emitting negative padding values,
 which are illegal

---
 crates/engine_spx2html/src/emission.rs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/crates/engine_spx2html/src/emission.rs b/crates/engine_spx2html/src/emission.rs
index c773755ae..a034e221c 100644
--- a/crates/engine_spx2html/src/emission.rs
+++ b/crates/engine_spx2html/src/emission.rs
@@ -1172,6 +1172,12 @@ impl EmittingState {
 
         let element = self.create_elem(element, true, common);
 
+        // Negative padding values are illegal.
+        let pad_left = match -x_min_tex as f32 * self.rems_per_tex {
+            pl if pl <= 0.0 => 0.0,
+            pl => pl,
+        };
+
         write!(
             self.content,
             "<{} class=\"canvas {}\" style=\"width: {}rem; height: {}rem; padding-left: {}rem{}\">",
@@ -1179,7 +1185,7 @@ impl EmittingState {
             layout_class,
             (x_max_tex - x_min_tex) as f32 * self.rems_per_tex,
             (y_max_tex - y_min_tex) as f32 * self.rems_per_tex,
-            -x_min_tex as f32 * self.rems_per_tex,
+            pad_left,
             valign,
         )
         .unwrap();