We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Describe the bug Seems that #848 is fixed only partially, for simple expressions - the fixing commit splits by '; ' and checks for .end presence via endsWith() , and follow-up changes drag that limitation too: https://github.com/svg/svgo/blame/main/plugins/prefixIds.js#L253
'; '
.end
endsWith()
For the following example, being a fragment of animation of rain drops, coming from https://github.com/basmilius/weather-icons (partly-cloudy-day-rain.svg):
<animate id="y1" attributeName="opacity" begin="0s; y1.end+.33s" dur=".67s" keyTimes="0; .25; 1" values="0; 1; 0"/>
the result is:
<animate id="partly-cloudy-day-rain_svg__j" attributeName="opacity" begin="0s; j.end+.33s" dur=".67s" keyTimes="0; .25; 1" values="0; 1; 0"></animate>
To Reproduce Steps to reproduce the behavior:
Expected behavior
<animate id="partly-cloudy-day-rain_svg__j" attributeName="opacity" begin="0s; partly-cloudy-day-rain_svg__j.end+.33s" dur=".67s" keyTimes="0; .25; 1" values="0; 1; 0"></animate>
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Describe the bug
Seems that #848 is fixed only partially, for simple expressions - the fixing commit splits by
'; '
and checks for.end
presence viaendsWith()
, and follow-up changes drag that limitation too:https://github.com/svg/svgo/blame/main/plugins/prefixIds.js#L253
For the following example, being a fragment of animation of rain drops, coming from https://github.com/basmilius/weather-icons (partly-cloudy-day-rain.svg):
the result is:
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: