fig cap location not changing in revealjs

[vrbiki]

Bug description

When I try to place the figure caption at the top of my figure it still defaults to the bottom.

Steps to reproduce

---
title: "test fig cap"
format:
  revealjs:
    fig-cap-location: top
editor: visual
---

## Slide

```{r}
#| fig-cap: "This caption should appear on top but doesn't"
#| fig-cap-location: top
#| label: fig-whatever

plot(1:10)
```

Does reference @fig-whatever help?

Expected behavior

By adding fig-cap-location: top to the code chunk (I also added it to the YAML in case the global declaration would help) I would have expected the caption to appear above the rendered plot.

Actual behavior

Instead it appears below.

Your environment

• IDE RStudio Version 2023.12.1+402 (2023.12.1+402)
• MacOS 14.3.1 (23D60)

Quarto check output

Quarto 1.4.549

[mcanouil]

Thanks for the report!

Could you properly format your post using code blocks for code and terminal outputs? Thanks.
If your code contains code blocks, you need to enclose it using more backticks, i.e., usually four ````.

---

You can share a self-contained "working" (reproducible) Quarto document using the following syntax, i.e., using more backticks than you have in your document (usually four ````).
See https://quarto.org/bug-reports.html#small-is-beautiful-aim-for-a-single-document-with-10-lines.

If you have multiple files (and if it is absolutely required to have multiple files), please share as a Git repository.

R	Python
````qmd --- title: "Reproducible Quarto Document" format: html engine: knitr --- This is a reproducible Quarto document. ```{r} x <- c(1, 2, 3, 4, 5) y <- c(1, 4, 9, 16, 25) plot(x, y) ```  The end. ````	````qmd --- title: "Reproducible Quarto Document" format: html engine: jupyter --- This is a reproducible Quarto document. ```{python} import matplotlib.pyplot as plt x = [1, 2, 3, 4, 5] y = [1, 4, 9, 16, 25] plt.plot(x, y) plt.show() ```  The end. ````

Additionally and if not already given, please share the output of quarto check within a code blocks (i.e., using three backticks ```txt), see https://quarto.org/bug-reports.html#check.

[cscheid]

fig-cap-location is currently only picked up for crossreferenceable figures. Adding label: fig-youridhere should solve the problem in the meantime.

[vrbiki]

Adding a label does not seem to do the trick

---
title: "test fig cap"
format:
  revealjs:
    fig-cap-location: top
editor: visual
---

## Slide

```{r}
#| fig-cap: "This caption should appear on top but doesn't"
#| fig-cap-location: top
#| label: fig-whatever

plot(1:10)
```

Does reference @fig-whatever help?

[cscheid]

Oh no, I'm so sorry! That's definitely a bug on our side (I've reproed and it's specific to revealjs).

(Incidentally, in html you don't need labe: fig-whatever either!)

[cscheid]

@cderv I think the bug here is that the revealjs post-processor doesn't respect the ordering between figcaption and the content inside the figure. I say that because the output we generate from Pandoc does put the caption on top:

          - t: "Div"
            attr: "('fig-whatever', ['quarto-figure', 'quarto-figure-center', 'quarto-float'], ['cap-location,top'])"
            content:
              - t: "RawBlock"
                format: "html"
                text: "<figure class=\"quarto-float quarto-float-fig\">"
              - t: "RawBlock"
                format: "html"
                text: "<figcaption class=\"quarto-float-caption-top quarto-float-caption quarto-float-fig\" id=\"fig-whatever-caption-0ceaefa1-69ba-4598-a22c-09a6ac19f8ca\">"
              - t: "Plain"
                content: "Figure 1: This caption should appear on top but doesn�t"
              - t: "RawBlock"
                format: "html"
                text: "</figcaption>"
              - t: "Div"
                attr: "('', [], ['aria-describedby,fig-whatever-caption-0ceaefa1-69ba-4598-a22c-09a6ac19f8ca'])"
                content:
                  - t: "Plain"
                    content:
                      - t: "Image"
                        attr: "('', [], ['width,960'])"
                        caption: []
                        src: "8938_files/figure-revealjs/fig-whatever-1.png"
                        title: ""
              - t: "RawBlock"
                format: "html"
                text: "</figure>"

It seems like it should be easy to fix because we're still emitting cap-location: top on the div.