monobackgroundcolor being ignored

[harrelfe]

I can't get monobackgroundcolor to take effect as described in https://quarto.org/docs/output-formats/html-themes.html#sass-variables. Here is a reproducible example.

---
title: "Test"
format:
  html:
    monobackgroundcolor: red
    backgroundcolor: cornsilk
    self-contained: true
    toc: true
    toc-location: left
    toc-title: Contents	
---

Here is some code: `xvariable <- pi`.  Regular text follows.

[cderv]

We correctly map the variable to the bootstrap rule, but this will only have effect on non-computation code blocks, from the comment

quarto-cli/src/resources/formats/html/bootstrap/_bootstrap-rules.scss

Lines 870 to 881 in 60edc74

	// Maps the pandoc 'monobackgroundcolor' to bootstrap
	// Note this only targets code outside of sourceCode blocks
	@if variable-exists(mono-background-color) {
	p code:not(.sourceCode),
	li code:not(.sourceCode),
	kbd,
	pre:not(.sourceCode),
	samp {
	background-color: $mono-background-color;
	padding: 0.2em;
	}
	}

So it should indeed work for the inline code, but it gets override by the rule right after

quarto-cli/src/resources/formats/html/bootstrap/_bootstrap-rules.scss

Lines 883 to 889 in 60edc74

	// Default padding if background is set
	p code:not(.sourceCode),
	li code:not(.sourceCode),
	td code:not(.sourceCode) {
	@if variable-exists(code-bg) {
	background-color: $code-bg;
	}

Both targets p code:not(.sourceCode)

@dragonstyle a new issue ?
Associated commits are months apart so maybe we missed a check ?

Or is this something expected depending on the theme ?

[dragonstyle]

This definitely seems it should work, so I'm transferring this to an issue!

[cderv]

Awesome. Thanks @harrelfe for the report ! This should be fixed in next pre-release !

[harrelfe]

That fixed it. Thank you!!

[ondrej-such]

How was this fixed?

I just ran harrelfe's example in quarto 1.3.450 and there was no background color in pdf created by XeLaTeX.

[ondrej-such]

Oh, it works when converting to HTML. I guess one needs to redefine a LaTeX macro to get it to work.