`#| out/fig-width:` ignored when specifying `#| layout-nrow/col:`

[HelenaLC]

Bug description

repex

.qmd:

```{r}
url1 <- "https://www.haustier-news.de/welpen/wp-content/uploads/2020/09/pomeranian.jpg"
url2 <- "https://www.allthingsdogs.com/wp-content/uploads/2020/01/White-Pomeranian-Feature-678x381.jpg"
```

# this works

```{r}
#| label: a-pomeranian
#| out-width: 50%
knitr::include_graphics(url1)
```

```{r}
#| label: another-pomeranian
#| out-width: 25%
knitr::include_graphics(url1)
```

# this doesn't

```{r}
#| label: two-pomeranians
#| fig-show: hold
#| out-width: 25%
#| layout-nrow: 1
knitr::include_graphics(url1)
knitr::include_graphics(url2)
```

output:

software

• Quarto version 1.1.168
• macOS Monterey version 12.2
• RStudio IDE version "Prairie Trillium" Release
  (1db809b8, 2022-05-16) for macOS

Checklist

• Please include a minimal, fully reproducible example in a single .qmd file? Please provide the whole file rather than the snippet you believe is causing the issue.
• Please format your issue so it is easier for us to read the bug report.
• Please document the RStudio IDE version you're running (if applicable), by providing the value displayed in the "About RStudio" main menu dialog?
• Please document the operating system you're running. If on Linux, please provide the specific distribution.

[cscheid]

@HelenaLC Thanks for the report! I want to gather some more information here. What did you expect the behavior to be when using these two options (layout-nrow and out-width) that (to my eyes) give conflicting information?

[zenggyu]

This issue also happens to out-height. I expected the height of the whole image group (all sub-images combined) to be half of what it would be if I didn't specify out-height: "50%".

[HelenaLC]

Similar to @zenggyu’s comment: My expectation (well, goal actually) was for the overall figure (of multiple panels) to span n%.
To give an example: I am including many figures that have 2-4 subpanels. Often, I’d like these to span 33% or 50% (margin) width only, e.g., when layout-ncol: 1-2, to avoid overly gigantic figures.

[cscheid]

Thanks for the clarification, I now see what you mean. The reason this was confusing is that, from my understanding, you're asking for slightly different semantics for out-height and out-width depending on whether we're emitting a single figure or multiple sub-figures. It's a totally reasonable expectation, but not how the code is set up right now (we're assuming users will want one or the other).

Unfortunately, this is actually fairly tricky to fix in the present version of the code, and we're starting a big refactoring of the figure infrastructure. I'm going to keep this open and we will definitely fix it, but the timeline is closer to 3-5 months away.

[jtr13]

Here's a MWE. This code produces the expected behavior if the file extension is .Rmd but doesn't work as expected if the file extension is changed to .qmd

Rendered from .Rmd file to .html

#| out-width: 40%
#| echo: false
plot(1, 2)
plot(3, 4)

Output:

Same code rendered from .qmd file to .html

Output:

[cscheid]

Moving this to 1.4 for our crossref overhaul.

(@jtr13 Just FYI, this behavior change you're seeing is not the same as the original issue posted. In quarto, figures flow as blocks by default, so you need to add layout-ncol: 2 to the code cell options. Note also that quarto will responsively reflow your columns in narrow formats, so you need to test the result on a wide screen)

[jtr13]

@cscheid Thanks for pointing that out.