Skip to content

Add specific icons to admonitions#12042

Open
Cykyrios wants to merge 1 commit into
godotengine:masterfrom
Cykyrios:admonition-icons
Open

Add specific icons to admonitions#12042
Cykyrios wants to merge 1 commit into
godotengine:masterfrom
Cykyrios:admonition-icons

Conversation

@Cykyrios

@Cykyrios Cykyrios commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Since #12017 adds icons to the proposed "class reference admonitions", this PR reuses the same icons and applies them to Note, Warning, Tip, and Important admonitions. All other admonitions (Danger, See also, Here be dragons, and anything else I missed) are left untouched.

This also changes the Important color from green (same as Tip) to yellow (same as Warning/Attention), for consistency in the icon color.

Master This PR
image image
More images In the following image, only the Important admonition is affected, the other two are identical to the master branch: image

More examples:
image
image

Depends on #12017.

@Calinou Calinou added enhancement content:website Issues related to adding website features and fixing bugs, whether on the front or back-end labels Jun 8, 2026

@Calinou Calinou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:before should be ::before instead (both work, but the former is deprecated).

@Cykyrios Cykyrios force-pushed the admonition-icons branch from d2ec901 to 0731ebd Compare June 9, 2026 04:45

@Calinou Calinou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code and design look good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content:website Issues related to adding website features and fixing bugs, whether on the front or back-end enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants