-
Notifications
You must be signed in to change notification settings - Fork 712
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
typescript(vx-heatmap): re-write package in TypeScript #520
Conversation
| 'y' | ||
| 'fill' | ||
| 'fillOpacity' | ||
>) { |
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.
nit: might be good to move component props type to the separate variable, something like ComponentProps
or OwnProps
for the better readability
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 call, done
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.
looks good 👌
7cb62f0
to
75e1f7e
Compare
🚀 Enhancements
This PR builds off #488 which introduces Typescript build config, and re-writes the
@vx/heatmap
package in TypeScript.Closes #519.
💥 Breaking Changes
React.Fragment
s, which requires bumping thepeerDep
forreact
to^16.3.0-0
Tests
/heatmaps
demo.d.ts
files@hshoff @schillerk @milesj @kristw @Rudeg