search

nagix / chartjs-plugin-style

Chart.js plugin for more styling options
MIT License
67 stars 19 forks source link

Remove trend line glow #6

Closed cohenu closed 5 years ago

cohenu commented 5 years ago

Hey :)

I've added the following to my chart:

outerGlowWidth: 10,
outerGlowColor: 'rgba(102, 85, 32, 1)',

and it seems like I have also hidden trend line drawn on my chart since the glow has been added to this line too and make it visible. is there any way to remove it ?

Thank you, merry christmas and happy new year!

nagix commented 5 years ago

Are you showing the trend line as a separate dataset from the main lines/bars? In that case, you can just set outerGlowWidth to 0 or outerGlowColor to 'transparent' for the trend line dataset.

cohenu commented 5 years ago

No, actually I have only one dataset (line chart), and once I activate the glow it seems like I have another transparent line but I cannot find from where this trend line coming from.

nagix commented 5 years ago

Can you share JSFiddle, Codepen or screenshot?

cohenu commented 5 years ago

link for screenshot: https://i.imgur.com/sNweRB5.jpg

nagix commented 5 years ago

This is clearly a bug in the outer glow function in line charts. I will fix it shortly.

cohenu commented 5 years ago

Thank you nagix very appreciated!

nagix commented 5 years ago

This issue has been fixed in 6b99442.

cohenu commented 5 years ago

Hi nagix :) thank you for the fix! I've just tested it and still have the same issue, see picture: https://i.imgur.com/EK3tLuX.png (using version 0.3.2)

nagix commented 5 years ago

Did you build a js file from the master branch? The fix is not merged into the release branch yet and will be included in v0.3.3 or v0.4.0 release.

cohenu commented 5 years ago

No I didn't, Sorry I didn't know. THANKS AGAIN! :)

nagix commented 5 years ago

Closing as v0.4.0 was just released.