Skip to content

feat(text): added invalid option to Text and TextArea wrappers#125

Merged
100stacks merged 2 commits intoHelixDesignSystem:masterfrom
abrande:text-add-invalid
Jan 18, 2021
Merged

feat(text): added invalid option to Text and TextArea wrappers#125
100stacks merged 2 commits intoHelixDesignSystem:masterfrom
abrande:text-add-invalid

Conversation

@abrande
Copy link

@abrande abrande commented Jan 12, 2021

  • Added additional invalid and label options in storybook

Screen Shot 2021-01-12 at 1 10 40 PM

Screen Shot 2021-01-12 at 1 10 51 PM

@netlify
Copy link

netlify bot commented Jan 12, 2021

✔️ Deploy preview for helix-react ready!

🔨 Explore the source changes: 64f7f1c

🔍 Inspect the deploy logs: https://app.netlify.com/sites/helix-react/deploys/6005c1df8884dd00072c7506

😎 Browse the preview: https://deploy-preview-125--helix-react.netlify.app

{...(label && { label })}
{...(disabled && { disabled })}
{...(required && { required })}
{...(invalid && { invalid })}
Copy link
Contributor

Choose a reason for hiding this comment

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

There are a few component examples in this file, it might be good to update them all with an invalid state since they all support disabled, required, etc.

@100stacks 100stacks added this to the v1.5.0-alpha milestone Jan 18, 2021
@100stacks 100stacks self-requested a review January 18, 2021 19:09
Copy link
Member

@100stacks 100stacks left a comment

Choose a reason for hiding this comment

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

Great job!

Dev LGTM

@100stacks 100stacks changed the title added invalid option to Text/TextArea feat(text): added invalid option to Text and TextArea wrappers Jan 18, 2021
@100stacks 100stacks merged commit 7ec5393 into HelixDesignSystem:master Jan 18, 2021
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.

3 participants