ChartJS: Features - Githubissues

iceteagroup / wisej-demobrowser

6 stars 5 forks source link

Closed AlaaTrAce closed 2 years ago

AlaaTrAce commented 2 years ago

msedge_0zG7gPGQbG

LRufenacht commented 2 years ago

[ ] I would take out three of the charts from this "Features" page. It looks a little too busy IMO. The Pie chart values are overlapping.
[ ] Set a minimum Size for the demo.
[ ] The AlertBoxes look weird being shown from the MiddleRight part of the screen. Try BottomRight and add a title so there isn't excessive blank space.

Looks good!

AlaaTrAce commented 2 years ago

I would take out three of the charts from this "Features" page. It looks a little too busy IMO. The Pie chart values are overlapping.

What charts would you recommend to remove?

LRufenacht commented 2 years ago

Polar, donut, pie

LRufenacht commented 2 years ago

Or keep donut and see if that looks good.

AlaaTrAce commented 2 years ago

msedge_CgR6uAnlSf What do you think?

LRufenacht commented 2 years ago

You can close after fixing!