-
Notifications
You must be signed in to change notification settings - Fork 127
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
chore(circuit-ui): DSYS-823 | mark phone number & color inputs as stable #2731
Conversation
🦋 Changeset detectedLatest commit: ebd15ee The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Size Change: -102 B (-0.02%) Total Size: 638 kB
ℹ️ View Unchanged
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #2731 +/- ##
=======================================
Coverage 87.49% 87.50%
=======================================
Files 213 213
Lines 11926 11921 -5
Branches 1556 1554 -2
=======================================
- Hits 10435 10431 -4
+ Misses 1441 1440 -1
Partials 50 50
|
packages/circuit-ui/components/ColorInput/ColorInput.stories.tsx
Outdated
Show resolved
Hide resolved
a37c49a
to
973cdb2
Compare
973cdb2
to
a4c526f
Compare
a4c526f
to
e376cc9
Compare
e376cc9
to
9f5535e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! Just some minor grammar/formatting nitpicks
Oh, and please don't forget to add a PR description! :)
|
||
## Validations | ||
|
||
Use the `validationHint` prop to communication information about the state of the ColorInput component, along with one of the following validation props: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The validationHint
prop can also be used without any other validation props to inform users about requirements before they've started entering a value.
6f0e177
to
ebd15ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✨
Addresses DSYS-823
Purpose
The experimental PhoneNumberInput and ColorInput components were added in Circuit UI v8.9 and v8.13 respectively. Their APIs are finally and many bugs have been fixed, so they are ready to be marked as stable.
Approach and changes
Definition of done