search

Joelius300 / ChartJSBlazor

This library is a modification of the awesome ChartJs.Blazor library by mariusmuntean. It's supposed to add more functionality to the LineChart and generally make the library more complete.
Other
40 stars 6 forks source link

Updated descriptions, added missing links, etc. #62

Closed SeppPenner closed 5 years ago

SeppPenner commented 5 years ago

I updated the wiki and took the time to update the readme a bit as well. Feel free to check this and tell me if there's something missing / not ok in your opinion.

Joelius300 commented 5 years ago

Looks good.

For the Wiki we could already include the samples for reworked charts (line, pie, polar), however I think it's going to be hard to keep those up to date, that's something I forget really easily.
Also putting a bit more emphasis on you should read the chart.js docs to understand how each chart works would be a good idea since I don't think this library contains enough information to understand how the charts actually work (especially for more detailed options). This would mean putting it higher up in the Wiki and mentioning it in the readme as well (where you direct them to the wiki for more 'detailed' samples).

SeppPenner commented 5 years ago

For the Wiki we could already include the samples for reworked charts (line, pie, polar), however I think it's going to be hard to keep those up to date, that's something I forget really easily.

I would do that when all are done just to make sure that all are up-to-date. But if you insist, I can add the examples now already.

For the rest: Check the new commit and https://github.com/Joelius300/ChartJSBlazor/wiki/Getting-Started if it's sufficient, please :)

Joelius300 commented 5 years ago

I would do that when all are done just to make sure that all are up-to-date. But if you insist, I can add the examples now already.

No, that's fine. I think it's a good idea. I only mentioned it because you direct them to that page which is empty so they might be confused.