From 6d0ba2ac14a796d43cd40b24bbca178114b9d457 Mon Sep 17 00:00:00 2001 From: cobyfrombrooklyn-bot Date: Sun, 22 Feb 2026 20:44:05 -0500 Subject: [PATCH] Fix nested admonitions using wrong header color Use child combinator (>) instead of descendant selector for blockquote-tag-title color rules. Without this, a nested admonition inherits the title color of its parent admonition instead of using its own type's color. Added nested admonition test case (important > note) to the existing admonition test suite. Fixes #3032 --- crates/mdbook-html/front-end/css/general.css | 10 +++++----- .../markdown/admonitions/expected/admonitions.html | 8 ++++++++ .../admonitions/expected_disabled/admonitions.html | 8 ++++++++ .../testsuite/markdown/admonitions/src/admonitions.md | 6 ++++++ 4 files changed, 27 insertions(+), 5 deletions(-) diff --git a/crates/mdbook-html/front-end/css/general.css b/crates/mdbook-html/front-end/css/general.css index df8a3efe..91401e3b 100644 --- a/crates/mdbook-html/front-end/css/general.css +++ b/crates/mdbook-html/front-end/css/general.css @@ -371,23 +371,23 @@ dd > p { border-inline-start-color: var(--blockquote-caution-color); } -.blockquote-tag-note .blockquote-tag-title { +.blockquote-tag-note > .blockquote-tag-title { color: var(--blockquote-note-color); } -.blockquote-tag-tip .blockquote-tag-title { +.blockquote-tag-tip > .blockquote-tag-title { color: var(--blockquote-tip-color); } -.blockquote-tag-important .blockquote-tag-title { +.blockquote-tag-important > .blockquote-tag-title { color: var(--blockquote-important-color); } -.blockquote-tag-warning .blockquote-tag-title { +.blockquote-tag-warning > .blockquote-tag-title { color: var(--blockquote-warning-color); } -.blockquote-tag-caution .blockquote-tag-title { +.blockquote-tag-caution > .blockquote-tag-title { color: var(--blockquote-caution-color); } diff --git a/tests/testsuite/markdown/admonitions/expected/admonitions.html b/tests/testsuite/markdown/admonitions/expected/admonitions.html index 8bbf4c87..962e7d79 100644 --- a/tests/testsuite/markdown/admonitions/expected/admonitions.html +++ b/tests/testsuite/markdown/admonitions/expected/admonitions.html @@ -23,4 +23,12 @@

[!UNKNOWN] This is an unknown tag.

+
+
+

Important

+

This is an important admonition.

+
+

Note

+

This nested note should have its own color.

+
\ No newline at end of file diff --git a/tests/testsuite/markdown/admonitions/expected_disabled/admonitions.html b/tests/testsuite/markdown/admonitions/expected_disabled/admonitions.html index 3bb296ed..65c10471 100644 --- a/tests/testsuite/markdown/admonitions/expected_disabled/admonitions.html +++ b/tests/testsuite/markdown/admonitions/expected_disabled/admonitions.html @@ -23,4 +23,12 @@ This is a caution.

[!UNKNOWN] This is an unknown tag.

+
+
+

[!IMPORTANT] +This is an important admonition.

+
+

[!NOTE] +This nested note should have its own color.

+
\ No newline at end of file diff --git a/tests/testsuite/markdown/admonitions/src/admonitions.md b/tests/testsuite/markdown/admonitions/src/admonitions.md index 0bf6526f..bb5102ac 100644 --- a/tests/testsuite/markdown/admonitions/src/admonitions.md +++ b/tests/testsuite/markdown/admonitions/src/admonitions.md @@ -19,3 +19,9 @@ > [!UNKNOWN] > This is an unknown tag. + +> [!IMPORTANT] +> This is an important admonition. +> +> > [!NOTE] +> > This nested note should have its own color.