You are viewing a plain text version of this content. The canonical link for it is here.
Posted to batik-users@xmlgraphics.apache.org by Konstantin Bulenkov <kb...@jetbrains.com.INVALID> on 2020/10/07 20:25:56 UTC
offset attribute should be set to 0 if it is missing in the stop tag
Hello everyone,
Some time ago, I've submitted a PR at
https://github.com/apache/xmlgraphics-batik/pull/20 but had no attention
since then.
Many modern design tools like Figma and others do not put offset attribute
when generating an SVG image if offset equals 0. See
https://www.w3.org/TR/SVG/pservers.html#GradientStopAttributes
Batik fails with exception while trying to read such Ss. Other programs
successfully open these files.
The pull request sets offset to 0 when the attribute is missing.
The patch is available here
https://github.com/apache/xmlgraphics-batik/pull/20/commits/20ad4382a0a4613956d1e543002e81b235ad1561
or as a patch file
https://github.com/apache/xmlgraphics-batik/pull/20/commits/20ad4382a0a4613956d1e543002e81b235ad1561.patch
Please review and merge if possible.
With best regards,
Konstantin
Example of an icon:
<svg width="32" height="32" viewBox="0 0 32 32" fill="none"
xmlns="http://www.w3.org/2000/svg">
<path d="M5.80005 22.2L1.30005 18.7L3.90005 13.9L7.80005 15.2L5.80005
22.2Z" fill="url(#paint0_linear)"/>
<path d="M31 9L30.5 26.4L18.9 31L12.6 26.9L31 9Z" fill="#087CFA"/>
<path d="M31 9L24.9 15L17.8 5.7L21.6 1.5L31 9Z" fill="url(#paint1_linear)"/>
<path d="M12.6 26.9L3.4 30.3L5.3 23.5L7.8 15.2L1 12.9L5.3 1L15.2
2.2L24.5 14L24.8 15L12.6 26.9Z" fill="url(#paint2_linear)"/>
<path d="M25 7H7V25H25V7Z" fill="url(#paint3_linear)"/>
<path d="M15.7 21.9H9V23H15.7V21.9Z" fill="white"/>
<path d="M12.63
10.49V9.25H9.26001V10.49H10.2V14.76H9.26001V15.99H12.63V14.76H11.68V10.49H12.63Z"
fill="white"/>
<path d="M15.87 16.09C15.34 16.09 14.9 15.99 14.55 15.79C14.19 15.59
13.9 15.35 13.67 15.08L14.6 14.04C14.79 14.25 14.99 14.42 15.18
14.54C15.38 14.66 15.59 14.72 15.83 14.72C16.11 14.72 16.33 14.63
16.49 14.45C16.65 14.27 16.73 13.99 16.73
13.6V9.26001H18.25V13.67C18.25 14.07 18.2 14.42 18.09 14.72C17.98
15.02 17.82 15.27 17.62 15.47C17.41 15.68 17.16 15.83 16.86
15.94C16.56 16.04 16.23 16.09 15.87 16.09Z" fill="white"/>
<defs>
<linearGradient id="paint0_linear" x1="3.21765" y1="18.0954"
x2="18.1487" y2="17.3385" gradientUnits="userSpaceOnUse">
<stop offset="0.0910927" stop-color="#FC801D"/>
<stop offset="0.2312" stop-color="#B07F61"/>
<stop offset="0.4086" stop-color="#577DB3"/>
<stop offset="0.5334" stop-color="#1E7CE6"/>
<stop offset="0.5934" stop-color="#087CFA"/>
</linearGradient>
<linearGradient id="paint1_linear" x1="27.5676" y1="16.8208"
x2="22.4964" y2="1.6674" gradientUnits="userSpaceOnUse">
<stop stop-color="#FE2857"/>
<stop offset="0.0784" stop-color="#CB3979"/>
<stop offset="0.1601" stop-color="#9E4997"/>
<stop offset="0.2474" stop-color="#7557B2"/>
<stop offset="0.3392" stop-color="#5362C8"/>
<stop offset="0.4365" stop-color="#386CDA"/>
<stop offset="0.5414" stop-color="#2373E8"/>
<stop offset="0.6576" stop-color="#1478F2"/>
<stop offset="0.794" stop-color="#0B7BF8"/>
<stop offset="1" stop-color="#087CFA"/>
</linearGradient>
<linearGradient id="paint2_linear" x1="5.3142" y1="8.0695"
x2="24.0899" y2="39.1256" gradientUnits="userSpaceOnUse">
<stop stop-color="#FE2857"/>
<stop offset="0.0800816" stop-color="#FE295F"/>
<stop offset="0.2065" stop-color="#FF2D76"/>
<stop offset="0.3034" stop-color="#FF318C"/>
<stop offset="0.3846" stop-color="#EA3896"/>
<stop offset="0.5532" stop-color="#B248AE"/>
<stop offset="0.7923" stop-color="#5A63D6"/>
<stop offset="1" stop-color="#087CFA"/>
</linearGradient>
<linearGradient id="paint3_linear" x1="15.976" y1="2.6251" x2="15.976"
y2="18.7675" gradientUnits="userSpaceOnUse">
<stop offset="0.0842986" stop-color="#878585"/>
<stop offset="0.5377"/>
</linearGradient>
</defs>
</svg>
Re: offset attribute should be set to 0 if it is missing in the stop tag
Posted by Kirill Grouchnikov <ki...@gmail.com>.
Github repository appears to be the mirror of
https://svn.apache.org/viewvc/xmlgraphics/ where the "real" work is
happening, so probably it would be better to propose this patch in that
original location.
On Wed, Oct 7, 2020 at 4:26 PM Konstantin Bulenkov <kb...@jetbrains.com.invalid>
wrote:
> Hello everyone,
>
> Some time ago, I've submitted a PR at
> https://github.com/apache/xmlgraphics-batik/pull/20 but had no attention
> since then.
>
> Many modern design tools like Figma and others do not put offset attribute
> when generating an SVG image if offset equals 0. See
> https://www.w3.org/TR/SVG/pservers.html#GradientStopAttributes
>
> Batik fails with exception while trying to read such Ss. Other programs
> successfully open these files.
> The pull request sets offset to 0 when the attribute is missing.
>
> The patch is available here
> https://github.com/apache/xmlgraphics-batik/pull/20/commits/20ad4382a0a4613956d1e543002e81b235ad1561
>
> or as a patch file
> https://github.com/apache/xmlgraphics-batik/pull/20/commits/20ad4382a0a4613956d1e543002e81b235ad1561.patch
>
>
> Please review and merge if possible.
>
>
> With best regards,
>
> Konstantin
>
>
> Example of an icon:
>
> <svg width="32" height="32" viewBox="0 0 32 32" fill="none" xmlns="http://www.w3.org/2000/svg">
> <path d="M5.80005 22.2L1.30005 18.7L3.90005 13.9L7.80005 15.2L5.80005 22.2Z" fill="url(#paint0_linear)"/>
> <path d="M31 9L30.5 26.4L18.9 31L12.6 26.9L31 9Z" fill="#087CFA"/>
> <path d="M31 9L24.9 15L17.8 5.7L21.6 1.5L31 9Z" fill="url(#paint1_linear)"/>
> <path d="M12.6 26.9L3.4 30.3L5.3 23.5L7.8 15.2L1 12.9L5.3 1L15.2 2.2L24.5 14L24.8 15L12.6 26.9Z" fill="url(#paint2_linear)"/>
> <path d="M25 7H7V25H25V7Z" fill="url(#paint3_linear)"/>
> <path d="M15.7 21.9H9V23H15.7V21.9Z" fill="white"/>
> <path d="M12.63 10.49V9.25H9.26001V10.49H10.2V14.76H9.26001V15.99H12.63V14.76H11.68V10.49H12.63Z" fill="white"/>
> <path d="M15.87 16.09C15.34 16.09 14.9 15.99 14.55 15.79C14.19 15.59 13.9 15.35 13.67 15.08L14.6 14.04C14.79 14.25 14.99 14.42 15.18 14.54C15.38 14.66 15.59 14.72 15.83 14.72C16.11 14.72 16.33 14.63 16.49 14.45C16.65 14.27 16.73 13.99 16.73 13.6V9.26001H18.25V13.67C18.25 14.07 18.2 14.42 18.09 14.72C17.98 15.02 17.82 15.27 17.62 15.47C17.41 15.68 17.16 15.83 16.86 15.94C16.56 16.04 16.23 16.09 15.87 16.09Z" fill="white"/>
> <defs>
> <linearGradient id="paint0_linear" x1="3.21765" y1="18.0954" x2="18.1487" y2="17.3385" gradientUnits="userSpaceOnUse">
> <stop offset="0.0910927" stop-color="#FC801D"/>
> <stop offset="0.2312" stop-color="#B07F61"/>
> <stop offset="0.4086" stop-color="#577DB3"/>
> <stop offset="0.5334" stop-color="#1E7CE6"/>
> <stop offset="0.5934" stop-color="#087CFA"/>
> </linearGradient>
> <linearGradient id="paint1_linear" x1="27.5676" y1="16.8208" x2="22.4964" y2="1.6674" gradientUnits="userSpaceOnUse">
> <stop stop-color="#FE2857"/>
> <stop offset="0.0784" stop-color="#CB3979"/>
> <stop offset="0.1601" stop-color="#9E4997"/>
> <stop offset="0.2474" stop-color="#7557B2"/>
> <stop offset="0.3392" stop-color="#5362C8"/>
> <stop offset="0.4365" stop-color="#386CDA"/>
> <stop offset="0.5414" stop-color="#2373E8"/>
> <stop offset="0.6576" stop-color="#1478F2"/>
> <stop offset="0.794" stop-color="#0B7BF8"/>
> <stop offset="1" stop-color="#087CFA"/>
> </linearGradient>
> <linearGradient id="paint2_linear" x1="5.3142" y1="8.0695" x2="24.0899" y2="39.1256" gradientUnits="userSpaceOnUse">
> <stop stop-color="#FE2857"/>
> <stop offset="0.0800816" stop-color="#FE295F"/>
> <stop offset="0.2065" stop-color="#FF2D76"/>
> <stop offset="0.3034" stop-color="#FF318C"/>
> <stop offset="0.3846" stop-color="#EA3896"/>
> <stop offset="0.5532" stop-color="#B248AE"/>
> <stop offset="0.7923" stop-color="#5A63D6"/>
> <stop offset="1" stop-color="#087CFA"/>
> </linearGradient>
> <linearGradient id="paint3_linear" x1="15.976" y1="2.6251" x2="15.976" y2="18.7675" gradientUnits="userSpaceOnUse">
> <stop offset="0.0842986" stop-color="#878585"/>
> <stop offset="0.5377"/>
> </linearGradient>
> </defs>
> </svg>
>
>