Don't show cell-output-display div when output: asis

[cderv]

Follow up on #4703 and trying to solve #7600

Currently, we do still add

::: {.cell-output-display}

when outputing knit_asis class content

Opening this PR to see how this simple change is impacted our tests - I feel this is a little change that could have side effect, especially because in our R knitr code output: asis means results: asis and both are not decoupled.

Though this change match our doc (https://quarto.org/docs/computations/execution-options.html#raw-output)

When output: asis is specified none of Quarto’s standard enclosing divs will be included.

However, I can see too that output: asis as the lowest priority if other options are provided.

quarto-cli/src/resources/rmd/hooks.R

Lines 400 to 416 in f38a30e

	# if there is a label, additional classes, a forwardAttr, or a cell.cap
	# then the user is deemed to have implicitly overridden results = "asis"
	# (as those features don't work w/o an enclosing div)
	needCell <- isTRUE(nzchar(label)) ||
	length(classes) > 1 ||
	isTRUE(nzchar(forwardAttr)) ||
	isTRUE(nzchar(cell.cap))
	if (identical(options[["results"]], "asis") && !needCell) {
	x
	} else {
	paste0(
	options[["indent"]], "::: {",
	labelId(label), paste(classes, collapse = " ") ,forwardAttr, "}\n", x, "\n", cell.cap ,
	options[["indent"]], ":::"
	)
	}
	})

This change a bit what we discussed, especially regarding #7607

[cscheid]

I'm going to close this one, @cderv, since the fix is the opposite of what we thought.

[cderv]

Yep all good! Thank you