Skip to content
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

Gradient in SVG support broken #38

Closed
soerendomroes opened this issue Feb 4, 2022 · 4 comments · Fixed by #69
Closed

Gradient in SVG support broken #38

soerendomroes opened this issue Feb 4, 2022 · 4 comments · Fixed by #69
Labels
enhancement New feature or request klighd-core

Comments

@soerendomroes
Copy link
Member

It seems to be the case that the gradient in the SCCharts visualization is not correctly exported. At least, if the file is opened in inkscape this is not the case.

To check:

What happens if SCChart is exported to svg in KIELER and VSCode

Do they look the same?
How about opening them in inkscape
How about exporting them to pdf via inkscape cli?

@soerendomroes soerendomroes added the bug Something isn't working label Feb 4, 2022
@Eddykasp
Copy link
Contributor

To expand on this issue: the problem doesn't happen upon opening the svg in Inkscape (or other svg viewers), but only once certain operations in inkscape are performed. For example resizing the image triggers it. This may be a problem in Inkscape.

@Eddykasp
Copy link
Contributor

Issue for Inkscape

@Eddykasp
Copy link
Contributor

The issue is already known Issue 2956

A potential temporary fix could be to define the gradients differently:

If you replace gradientTransform="rotate(90)" with x1="0" x2="0" y1="0" y2="1" it appears to work fine

maybe @NiklasRentzCAU has some input on whether changing this in the rendering is worth the effort?

NiklasRentzCAU added a commit that referenced this issue Apr 27, 2022
…ehavior, but works around an issue in Inkscape when opening with that. Fixes #38
@NiklasRentzCAU
Copy link
Member

Thanks for that hint, #69 will work around the issue in Inkscape and use the gradient coordinates instead of the gradient angle definitions.

@NiklasRentzCAU NiklasRentzCAU added enhancement New feature or request klighd-core and removed bug Something isn't working labels Apr 27, 2022
NiklasRentzCAU added a commit that referenced this issue Apr 28, 2022
…ehavior, but works around an issue in Inkscape when opening with that. Fixes #38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request klighd-core
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants