Wrap in nonincremental div code annotation definition list to prevent incremental applying

[cderv]

fixes #7142

When we deal with Code Annotation in our Lua processing, if incremental: true is set, then we make sure to wrap the definition list in a Div of class nonincremental, which is the way to opt out per Pandoc's documentation: https://pandoc.org/MANUAL.html#incremental-lists

For now, it will modify the resulting DOM in revealjs as Pandoc removes the class, but does not clean div when it is empty: jgm/pandoc#10328

We could do the cleaning if necessary. We could generalise the scaffold added for blockquotes

quarto-cli/src/format/reveal/format-reveal.ts

Lines 883 to 902 in 820c998

	// Remove blockquote scaffolding added in Lua post-render to prevent Pandoc syntax for applying
	if (doc.querySelectorAll("div.blockquote-list-scaffold")) {
	const blockquoteListScaffolds = doc.querySelectorAll(
	"div.blockquote-list-scaffold",
	);
	for (const blockquoteListScaffold of blockquoteListScaffolds) {
	const blockquoteListScaffoldEL = blockquoteListScaffold as Element;
	const blockquoteListScaffoldParent =
	blockquoteListScaffoldEL.parentNode;
	if (blockquoteListScaffoldParent) {
	while (blockquoteListScaffoldEL.firstChild) {
	blockquoteListScaffoldParent.insertBefore(
	blockquoteListScaffoldEL.firstChild,
	blockquoteListScaffoldEL,
	);
	}
	blockquoteListScaffoldParent.removeChild(blockquoteListScaffoldEL);
	}
	}
	}

This is one way to solve #7142 - As mention in Team discussion, we could also do post processing to remove fragment on DefinitionList created for annotation.

Design question:

For now, this applies to all code annotation. This means that when incremental: true and code-annoation: below, the definition list won't be incremental.

We could only apply this logic to code-annotation: hover or code-annotation: select if we think those list should be incremental like any other Definition List. I am hesitant on this...

Enregistrement.2024-10-23.163853.mp4

Any thoughts ?

[cderv]

We keep current behavior and so only fix this for hover and select and keep incremental for below

[cderv]

For now, it will modify the resulting DOM in revealjs as Pandoc removes the class, but does not clean div when it is empty: jgm/pandoc#10328

jgm/pandoc#10328 has been solved in upstream pandoc so the <div> will be removed in 1.7 when we'll update Pandoc.