Merge pull request #1029 from quarto-dev/reveal-stretch-aside

[reveal] do not apply auto-stretch if aside is used

Author: jjallaire

src/format/reveal/format-reveal.ts

@@ -645,15 +645,17 @@ function applyStretch(doc: Document, autoStretch: boolean) {
       const image = images[0];
       const imageEl = image as Element;
 
-      // screen out early specials divs (layout panels, columns, fragments, ...)
       if (
+        // screen out early specials divs (layout panels, columns, fragments, ...)
         findParent(imageEl, (el: Element) => {
           return el.classList.contains("column") ||
             el.classList.contains("quarto-layout-panel") ||
             el.classList.contains("fragment") ||
             el.classList.contains(kOutputLocationSlide) ||
             !!el.className.match(/panel-/);
-        })
+        }) ||
+        // Do not autostrech if an aside is used
+        slideEl.querySelectorAll("aside").length !== 0
       ) {
         continue;
       }

tests/docs/reveal/stretch.qmd

@@ -234,4 +234,12 @@ Some content
 plot(cars)
 ```
 
+:::
+
+## With aside on slide  {#aside}
+
+![Caption](https://revealjs.com/images/logo/reveal-black-text.svg)
+
+::: {.aside}
+Something here as an aside
 :::

tests/smoke/render/render-reveal.test.ts

@@ -66,5 +66,6 @@ testRender(input, "revealjs", false, [
     "#custom-divs-caption img.r-stretch",
     "#custom-divs-knitr img.r-stretch",
     "#custom-divs-knitr-caption img.r-stretch",
+    "#aside img.r-stretch",
   ]),
 ]);