timqian/chart.xkcd

Do you want to work on this issue?

You can request for a bounty in order to promote it!

[new chart type]: combined chart #22

Dkalugin posted onGitHub

Hello! Can I build a combined chart like a screenshot? https://tetis-capital.ru/upload/image/chart.png


@issuehunt has funded $50.00 to this issue.


posted by issuehunt-app[bot] over 5 years ago

So, I'm planning to make a new Combined.js in the src and then build it as a separate new chart or you want me to re-use the Bar and Line code, is that possible in some way?

posted by gurrrung over 5 years ago

@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. https://github.com/timqian/chart.xkcd/issues/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

Ref: How chart.js implement mixed type chart https://www.chartjs.org/docs/latest/charts/mixed.html

posted by timqian over 5 years ago

Fund this Issue

$50.00
Funded
Only logged in users can fund an issue

Pull requests

Recent activities

issuehunt funded 50.00 for timqian/chart.xkcd# 22
over 5 years ago