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

Fix: xlink:href value format. #455

Merged
merged 2 commits into from May 16, 2019
Merged

Conversation

H1Gdev
Copy link
Contributor

@H1Gdev H1Gdev commented May 16, 2019

Format of xlink:href value is invalid.
So, output SVG file is not rendered on browsers.

For example, part of pservers-grad-01-b.svg SVG file output.

  • Input
    <linearGradient id="Grad1b" xlink:href="#Grad1a"/>
  • Output
    <linearGradient y2="0%" gradientUnits="objectBoundingBox" xlink:href="url(#Grad1a)" id="Grad1b" xml:space="default" />
  • Expected
    <linearGradient y2="0%" gradientUnits="objectBoundingBox" xlink:href="#Grad1a" id="Grad1b" xml:space="default" />

Copy link
Member

@mrbean-bremen mrbean-bremen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for another fix! I will have a closer look tonight, but I noticed that there are 2 open issues releated to different aspects of xlink:href: #71 and #220. Both have some code attached, which is related to the same location. If you get the time - could you please look if it makes sense to integrate these?

@H1Gdev
Copy link
Contributor Author

H1Gdev commented May 16, 2019

@mrbean-bremen

I will consider separately.
Do you have data for testing for #71 and #220 to this day ?

@mrbean-bremen
Copy link
Member

Sorry, no - I guess we can just adapt an existing SVG to use these constructs.

@mrbean-bremen mrbean-bremen merged commit 9dd2d50 into svg-net:master May 16, 2019
@H1Gdev H1Gdev deleted the gradient branch May 16, 2019 21:05
H1Gdev added a commit to H1Gdev/SVG that referenced this pull request Aug 27, 2019
H1Gdev added a commit to H1Gdev/SVG that referenced this pull request Aug 28, 2019
- It doesn't work properly with certain IRIs, so fix it in another way.
- ref. svg-net#455.
H1Gdev added a commit to H1Gdev/SVG that referenced this pull request Sep 2, 2019
- It doesn't work properly with certain IRIs, so fix it in another way.
- ref. svg-net#455.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants