Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(root): add guidance for neutral alert and show default icon prop #1171

Merged

Conversation

lz405
Copy link
Contributor

@lz405 lz405 commented Oct 17, 2024

Summary of the changes

Add guidance for neutral alert and show default icon prop.

Related issue

Issue #1126

Checklist

Copy link

@lz405 lz405 force-pushed the guidance/1126-add-guidance-for-neutral-alert branch from b9f364d to b4a924a Compare October 17, 2024 12:40
ad9242
ad9242 previously requested changes Oct 17, 2024
src/content/structured/components/alerts/code.mdx Outdated Show resolved Hide resolved
src/content/structured/components/alerts/code.mdx Outdated Show resolved Hide resolved
@lz405 lz405 force-pushed the guidance/1126-add-guidance-for-neutral-alert branch from b4a924a to 2d0d4ab Compare October 18, 2024 08:33
add examples for neutral variant alerts and alerts with the    show-default-icon prop set to false

fix #1126
@lz405 lz405 force-pushed the guidance/1126-add-guidance-for-neutral-alert branch from 2d0d4ab to e886f1f Compare October 18, 2024 08:59
Copy link
Contributor

@GCHQ-Developer-847 GCHQ-Developer-847 left a comment

Choose a reason for hiding this comment

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

Noticed the issue with the margin when showDefaultIcon is false but I know it will be fixed by PR #2616 so have approved :)

@GCHQ-Developer-299 GCHQ-Developer-299 merged commit 1ccf6ae into develop Nov 7, 2024
3 checks passed
@GCHQ-Developer-299 GCHQ-Developer-299 deleted the guidance/1126-add-guidance-for-neutral-alert branch November 7, 2024 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants