Skip to content

fix(#3123): docs update - badge icon type prop#450

Merged
ArakTaiRoth merged 1 commit intoGovAlta:alphafrom
syedszeeshan:Syed/3123-docs
Nov 13, 2025
Merged

fix(#3123): docs update - badge icon type prop#450
ArakTaiRoth merged 1 commit intoGovAlta:alphafrom
syedszeeshan:Syed/3123-docs

Conversation

@syedszeeshan
Copy link
Collaborator

No description provided.

@netlify
Copy link

netlify bot commented Nov 5, 2025

Deploy Preview for abgov-ui-component-docs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 6ad0ed7
🔍 Latest deploy log https://app.netlify.com/projects/abgov-ui-component-docs/deploys/6916530692e61e00078384d8
😎 Deploy Preview https://deploy-preview-450--abgov-ui-component-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@syedszeeshan syedszeeshan marked this pull request as ready for review November 5, 2025 16:58
@syedszeeshan syedszeeshan linked an issue Nov 5, 2025 that may be closed by this pull request
},
{
label: "Icon type",
type: "string",
Copy link
Collaborator

Choose a reason for hiding this comment

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

You have this set to a type of String, it would probably be better to treat this like we do Trailing and Leading Icon properties in other components.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

PR updated.

image

@ArakTaiRoth ArakTaiRoth merged commit 45b3067 into GovAlta:alpha Nov 13, 2025
4 checks passed
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.

icon property doesn't default to false

3 participants