-
-
Notifications
You must be signed in to change notification settings - Fork 662
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add fillShadowGradientTo option #513
Conversation
How we can help to move this PR forward? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Any chance this can be merged? |
Bump |
Why hasn't this been merged? |
it's coming up on a year since this PR was submitted back in May of 2021... I believe perhaps this project has been abandoned unfortunately. @Devorein @Hermanya @noktigula @sreuter anyone have any further information for us? |
Thank you for your contribution 👍 this is now available in v6.12.0 |
In an effort to allow for a linear gradient shadow fill that blends one color into another, this PR replaces
fillShadowGradient
andfillShadowGradientOpacity
with new "To" and "From" options:fillShadowGradientFrom
fillShadowGradientFromOpacity
fillShadowGradientTo
fillShadowGradientToOpacity
In the
AbstractChart
component, iffillShadowGradientTo
is provided in the chart config, then it's used for the second<Stop>
SVG element, otherwisefillShadowGradientFrom
is used. Similarly, iffillShadowGradientToOpacity
is provided, then that's used for the second<Stop>
SVG element's opacity, otherwise a default value of 0 is used.Lastly, it maintains backwards compatibility with the current
fillShadowGradient
andfillShadowGradientOpacity
options.