adrianhajdin / project_syncfusion_dashboard

This is a code repository for the corresponding YouTube video. In this tutorial we are going to build and deploy a an admin dashboard app using React.js and Syncfusion
https://jsmastery.pro
GNU Affero General Public License v3.0
3.08k stars 1.05k forks source link

Sparkline Component doesn't work #17

Closed doums10 closed 2 years ago

doums10 commented 2 years ago

I dont have the Sparkline line on the Ecommerce view, anyone knows why?

Capture d’écran 2022-09-13 à 17 39 14 Capture d’écran 2022-09-13 à 17 39 59
Pannu786 commented 2 years ago

Change yName='y' to yName='yval' This should fix the issue.

CGTones commented 2 years ago

i couldn't get sparkline to render until i refactored it to a class component as shown here in the repo:

https://github.com/adrianhajdin/project_syncfusion_dashboard/blob/main/src/components/Charts/SparkLine.jsx

Pannu786 commented 2 years ago

Can show us how was your function-based code?

On Fri, 16 Sep 2022, 11:54 Trevor Jones, @.***> wrote:

i couldn't get sparkline to render until i refactored it to a class component as shown here in the repo:

https://github.com/adrianhajdin/project_syncfusion_dashboard/blob/main/src/components/Charts/SparkLine.jsx

— Reply to this email directly, view it on GitHub https://github.com/adrianhajdin/project_syncfusion_dashboard/issues/17#issuecomment-1248965633, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKO6XDW6FTOLKURYWK4FJXTV6QHB7ANCNFSM6AAAAAAQL24AZM . You are receiving this because you commented.Message ID: @.*** com>

CGTones commented 2 years ago

sry. my mistake. the issue i had was only triggered when resizing the browser window. nothing would display on resize. resolved upon refactoring to pure class component. but fwiw, here is the link to the commit history on my alt gh account

https://github.com/degenGenesis/dashboard/commit/b4dd1cee1276b251a4ab2d340731c454d03ca6f9

doums10 commented 2 years ago

Change yName='y' to yName='yval' This should fix the issue.

yes, its correct, thanks bro