-
-
Notifications
You must be signed in to change notification settings - Fork 200
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
[new chart type]: combined chart #22
Comments
@issuehunt has funded $50.00 to this issue.
|
So, I'm planning to make a new Combined.js in the |
@gurrrung Thanks for the interest, I think we should re-use the Bar and Line code in some way, so we won't need to copy-paste code around when the charts changes(e.g. #27), but the current implementation of them is not easy to be reused, we may need to refactor the code a little Suggestions on the refactoring are welcomed
|
Hello!
Can I build a combined chart like a screenshot?
https://tetis-capital.ru/upload/image/chart.png
IssueHunt Summary
Backers (Total: $50.00)
Become a backer now!
Or submit a pull request to get the deposits!
Tips
IssueHunt has been backed by the following sponsors. Become a sponsor
The text was updated successfully, but these errors were encountered: