Inline code style for admonition seems odd due to override

[polarathene]

• I've read the [contribution guidelines][1] and agree with them

I've found a bug and checked that ...

• ... the problem doesn't occur with the default MkDocs template
• ... the problem is not in any of my customizations (CSS, JS, template)
• ... the documentation does not mention anything about my problem
• ... there are no open or closed issues that are related to my problem

Description

I don't quite get the reasoning for this override?:

mkdocs-material/src/assets/stylesheets/main/extensions/markdown/_admonition.scss

Lines 154 to 163 in 414ffe1

	// Omit background on inline code blocks, as they don't go well with the
	// pastelly tones applied to admonition titles
	code {
	margin: initial;
	padding: initial;
	color: currentColor;
	background-color: transparent;
	border-radius: initial;
	box-shadow: none;
	}

How is the override visually better? If it was a concern for the user they might as well at that point just not use the inline code syntax?

If the concern is with contrast, wouldn't it be better to instead use alpha?

Expected behavior

EDIT: See the comparison images below in the 2nd message.

original examples

Any of the following:

1. Restore bg-color:
   

2. Full restore:
   

3. Change bg-color to brighter shade with opacity (#ffffffbf):
   

4. Change bg-color to darker shade with opacity (#3a3a3a30):
   

5. Change bg-color to better match admonition colour (#d7d7d7ad with CSS mix-blend-mode: multiply):
   

6. or with mix-blend-mode: luminosity:
   

Personally 3 or 6 look quite nice.

Actual behavior

Package versions / Project configuration / System information

Active release used for project demo site (if modified to use an inline code block in admonition title): https://squidfunk.github.io/mkdocs-material/reference/admonitions

[polarathene]

Replacing the referenced CSS rules to be only the 3 changes below with different blend modes looks much better personally. I've also included at the end how these look with current override, and override removed for comparison.

---

Difference

color: currentColor;
background-color: #bbbbbba3;
mix-blend-mode: difference;

---

Color-Burn

color: currentColor;
background-color: #444;
mix-blend-mode: color-burn;

---

Luminosity

color: currentColor;
background-color: #ccc;
mix-blend-mode: luminosity;

---

Multiply

color: currentColor;
background-color: #e8e8e8;
mix-blend-mode: multiply;

---

Just light bg with opacity:

background-color: #ffffffb7

---

All overrides removed:

---

All overrides applied (current upstream style)

[squidfunk]

Thanks for your suggestions.

If the concern is with contrast, wouldn't it be better to instead use alpha?

We've used alpha values for code block colors in the past, and it came with other problems, for example when using line numbers with the pymdownx-inline method, which some users prefer to the table-based solution. The code was shining through the line numbers when scrolling, so we had to switch to opaque colors.

IIRC (it's a long time ago), we removed the background because we've used alpha values, so since that's no longer true, we might add it back. After all, it's more consistent, and I'm fine with the look. With the alpha approach, it looked really bad.

[squidfunk]

Removed override in cb8d3f4.

[squidfunk]

Released as part of 7.0.7.