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
Chart QA tasks #115
Chart QA tasks #115
Conversation
1811cf1
to
4f6a882
Compare
packages/client/svelte/src/lib/internal/ui/chart/_Wrapper.svelte
Outdated
Show resolved
Hide resolved
f2d1b06
to
05f9301
Compare
b4f0fbd
to
9f02668
Compare
🦋 Changeset detectedLatest commit: 7649a21 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 |
5a7fc58
to
715fb72
Compare
docs/snippets/charts/common.mdx
Outdated
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.
Mintlify allow DRY docs.Having charts config duplicated in each chart type was going me crazy. Sorry the ocd but I think this is much better now
d248846
to
6e8c0c0
Compare
if (total === 0) { | ||
return [row[0]].concat(rowData.map(() => 0)) | ||
} | ||
if (total === 0) return row.map((val) => { |
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.
nitpick: add brackets to this if
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.
You're very right in this one. I think we should for by ESLint the brackets
packages/client/svelte/src/lib/internal/ui/chart/_Wrapper.svelte
Outdated
Show resolved
Hide resolved
6e8c0c0
to
97dca2f
Compare
- Add sort prop - Chart should have a title and description prop - Truncated X axis in mixed charts - Make the charts animate to the new state instead of flicker - Error display as in old latitude
97dca2f
to
7649a21
Compare
What?
Improve initial charts with after some initial QA
Issue
#143
TODO
<ChartLine />
columns can be defined withoutchartType
. By default areline
type.sort
prop and make clear that<LineChart />
is optional to passchartType
now