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(meter): add docs, update designs #3463
Conversation
Run & review this pull request in StackBlitz Codeflow. |
🦋 Changeset detectedLatest commit: 00db38c The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 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 |
✅ Deploy Preview for paste-theme-designer ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 00db38c:
|
Size Change: +217 B (0%) Total Size: 1.01 MB
ℹ️ View Unchanged
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 00db38c. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
21a8da9
to
59a6c3c
Compare
✅ Deploy Preview for paste-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Passing run #6764 ↗︎
Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
59a6c3c
to
ada46e7
Compare
ada46e7
to
5e905bb
Compare
06ee11f
to
b122859
Compare
value?: number; | ||
id: string; | ||
showValueLabel?: boolean; |
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.
question: this is a breaking api change. Are we comfortable marking these changes as a patch when the package is on 1.0.0?
I think it's fine, since the docs aren't even out. But just wanted to call it out
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.
good point, I agree that it's fine since nobody's using it and it is an alpha component
b122859
to
6a7ed21
Compare
6a7ed21
to
a215ffe
Compare
a215ffe
to
00db38c
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.
Great!
In this PR
Meter API
Design and API are as closely aligned as possible. Any aria values present in the DOM can be displayed on the Meter. The value label is set on the MeterLabel component using the
valueLabel
prop. Min and max value labels are optional and set on the Meter usingminLabel
andmaxLabel
props. This allows the consumer to have full control, using a string rather than a number.