Merge pull request #9725 from quarto-dev/bugfix/9724

lua,float - force-align Listing floats to left

Author: cscheid

news/changelog-1.5.md

@@ -196,6 +196,7 @@ All changes included in 1.5:
 - ([#9550](https://github.com/quarto-dev/quarto-cli/issues/9550)): Don't crash when subcaptions are incorrectly specified with `fig-subcap: true` but no embedded subcaptions.
 - ([#9652](https://github.com/quarto-dev/quarto-cli/pull/9652)): Allow `--output-dir` to refer to absolute paths in `quarto render`.
 - ([#9701](https://github.com/quarto-dev/quarto-cli/issues/9701)): Fix issue with callouts with non-empty titles that have no string content.
+- ([#9724](https://github.com/quarto-dev/quarto-cli/issues/9724)): Force-align text in floats of type `Listing` to the left.
 - Add support for `{{< lipsum >}}` shortcode, which is useful for emitting placeholder text. Provide a specific number of paragraphs (`{{< lipsum 3 >}}`).
 - Resolve data URIs in Pandoc's mediabag when rendering documents.
 - Increase v8's max heap size by default, to avoid out-of-memory errors when rendering large documents (also cf. https://github.com/denoland/deno/issues/18935).

src/resources/filters/customnodes/floatreftarget.lua

@@ -783,6 +783,11 @@ function float_reftarget_render_html_figure(float)
     pandoc.Attr("", {}, figure_attrs.figureAttr))
   if float.type == "Listing" then
     div.attr.classes:insert("listing")
+    -- in the special case of listings, we likely have text content
+    -- including annotations, which require left alignment
+    -- we hard-code this here.
+    -- https://github.com/quarto-dev/quarto-cli/issues/9724
+    figure_attrs.align = "left"
   end
   div.attr.classes:insert("quarto-float")
 

tests/docs/smoke-all/2024/05/20/9724.qmd

@@ -0,0 +1,13 @@
+---
+_quarto:
+  tests:
+    html:
+      ensureHtmlElements:
+        - ["div.quarto-figure-left"]
+        - []
+---
+
+```{.python filename="run_experiments.py" #lst-1 lst-cap="Corpus Loading/Generation Logic"}
+line 1
+line 2
+```