Quarto 1.4.533: `code-fold:true` together with the new chunk option `lst-label` does not work

[petzi53]

Bug description

Using code-fold: true together with the chunk option lst-label does not work.

But code-fold: true used together with the div syntax works.

Steps to reproduce

## First Test with lst-label

```{r}
#| lst-label: lst-first-test
#| lst-cap: "Test `code-fold: true` with lst-label"
#| code-fold: true

x <- 1
```


## Second Test with div syntax

::: {#lst-second-test}
```{r}
#| code-fold: true

x <- 1


```

Test `code-fold: true` with div syntax

:::

Expected behavior

In the first example, the code should be folded, exactly as in the second (correct) example.

Actual behavior

Your environment

• IDE: RStudio Version 2023.12.0+369 (2023.12.0+369)
• OS: macOS: Sonoma 14.2.1 (23C71)

Quarto check output

Quarto 1.4.349
[✓] Checking versions of quarto binary dependencies...
      Pandoc version 3.1.7: OK
      Dart Sass version 1.55.0: OK
      Deno version 1.33.4: OK
[✓] Checking versions of quarto dependencies......OK
[✓] Checking Quarto installation......OK
      Version: 1.4.349
      Path: /Applications/quarto/bin

[✓] Checking tools....................OK
      TinyTeX: (not installed)
      Chromium: (not installed)

[✓] Checking LaTeX....................OK
      Using: Installation From Path
      Path: /Library/TeX/texbin
      Version: 2023

[✓] Checking basic markdown render....OK

[✓] Checking Python 3 installation....OK
      Version: 3.10.9 (Conda)
      Path: /Users/petzi/anaconda3/bin/python
      Jupyter: 5.2.0
      Kernels: python3

[✓] Checking Jupyter engine render....OK

(\) Checking R installation...........R version 4.3.2 (2023-10-31)
[✓] Checking R installation...........OK
      Version: 4.3.2
      Path: /Library/Frameworks/R.framework/Resources
      LibPaths:
        - /Library/Frameworks/R.framework/Versions/4.3-x86_64/library
        - /Library/Frameworks/R.framework/Versions/4.3-x86_64/Resources/library
      knitr: 1.45
      rmarkdown: 2.25

[✓] Checking Knitr engine render......OK

[cderv]

Thanks for the report. I can reproduce

We get this because lst- triggers the block to be a FloatRefTarget() (https://quarto.org/docs/prerelease/1.4/lua_changes.html#the-floatreftarget-ast-node) in the AST.

By doing this, we remove the attributes to place them on the FloatRefTarget and no more the codeblock

quarto-cli/src/resources/filters/quarto-pre/parsefiguredivs.lua

Lines 569 to 576 in 27b29d9

	local attr = code.attr
	code.attr = pandoc.Attr("", {}, {})
	return quarto.FloatRefTarget({
	attr = attr,
	type = "Listing",
	content = { content },
	caption_long = caption_inlines,
	}), false

I don't think code-fold: true is thus handled on Code Listing
So we need probably to handle code-fold on FloatRefTarget now. 🤔

[cderv]

@petzi53 why close ? 🤔 Wrong intention ? I believe we need to fix this

[cscheid]

@cderv is right: the issue here is the fact that an executable code cell that is also a listing can also be foldable.

I propose we try to fix this in 1.5, since there's a simple workaround in 1.4, namely to split the code listing in two: One executable code cell with echo: false, and one non-executable code cell that folds. (This is how things were done in 1.3).

[petzi53]

Sorry for closing. It was my mistake.