search

dcasia / chartjs-widget

A ChartJs widget for laravel Nova Dashboard.
MIT License
8 stars 5 forks source link

Add Pie Chart #5

Closed xoco70 closed 1 year ago

xoco70 commented 4 years ago

This PR is not ready to merge.

I have included a Pie Chart which is rendering well.

Pie has still no custom options.

Difference between LineChartand BarChart is that both take several DataSet, but Pie needs only 1 DataSet.

When adding only 1 dataset, you will have same config / label for all quarters of you pie.

When adding 2 datasets, it will be displays as circles, that could be an option for third dimension pies, but it is not the main use case.

How do you want me to manage this for your repo ?

milewski commented 4 years ago

Hey if the dataset needs to be different depending on the chart i think its okay to have another one ... like PieDataSet BarDataSet LineDataSet etc... so then you could have custom data shapes for each of them

milewski commented 1 year ago

All charts type support added on latest version