Feature/pdfcodeblock

[kandolfp]

Description

As discussed in #11698, for PDF output, nested fenced code blocks no longer allow for page break and you can create color leakage. As this is an inherent problem with the LaTeX treatment of floating environments and the tcolorbox package (currently) used to create the background color for fenced code blocks in PDF documents a workaround is included.

The variable code-block-bg can be set to a HTML color tag or to a boolean. The boolean false has no particular effect as this will create no background color. Therefore, I selected this variable to explicitly deactivate the tcolorbox when set to false. The potential conflict with code-block-border-left is avoided by only applying the behavior if code-block-border-left is false or not set. Line numbers an annotations can be combined without any problems.

I included the example from the #11698 as a test as well but from my point of view this is not extremely helpful. I would need to test the resulting tex file for certain content and I could not find out how this is done. If you can hint me in the right direction I'm happy to extend this.

Note on implementation

This could also be implemented as a elseif but from my point of view it is cleaner this way, happy to change if you think differently.

I tried to argue why I reused a variable instead of creating a new variable but if this multi use is not wanted I can change it to a specific variable.

Miscellaneous

I also found two typos on the way. If I should create a separate PR from them please let me know.

Checklist

I have (if applicable):

• filed a contributor agreement.
• referenced the GitHub issue this PR closes -> Color leakage and no page break for .callout environments #11698
• updated the appropriate changelog in the PR
• ensured the present test suite passes
• added new tests -> see comment above for suitability
• created a separate documentation PR in Quarto's website repo and linked it to this PR -> PR

[cscheid]

Thanks for the PR! We should be able to merge something like it.

There are a few things we want to check. Specifically, Shaded is likely used by Pandoc in other scenarios and we want to make sure we're not breaking those. We'll get back to you soon.

[kandolfp]

Makes total sense! If I can be of help, please let me know.