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 ArgumentException in SvgTextBase.Bounds when has a whitespace line #963
Fix ArgumentException in SvgTextBase.Bounds when has a whitespace line #963
Conversation
Fix ArgumentException in SvgTextBase.Bounds when has tspan.text is only whitespace
add testsuite
correct indentation
change test file to plain svg
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! Can you please add your test file to PassingTests.csv (for the unit tests) and PassingTests.txt (for the test runner), and also add an entry to the release notes?
This process does not work with SVG file for this test. |
@mrbean-bremen Sorry, I don't know which release version I shoud update this pull request issue to. |
Ah, we just had a release - just put it on top of that, you may prepend Unreleased as a header. |
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.
Tnanks - also for the missing png, which I forgot to mention! If you add the release note, we can merge it.
…ace (#963) BuildProcessTemplates CONTRIBUTING.md Generators Nuget README.md Samples Source Tests doc docfx.json index.md license.txt Update SvgTextBase.Drawing.cs BuildProcessTemplates CONTRIBUTING.md Generators Nuget README.md Samples Source Tests doc docfx.json index.md license.txt add test and release notes
Fix ArgumentException in SvgTextBase.Bounds when a svgtext has a textspan having only whitespace.
Reference Issue
What does this implement/fix? Explain your changes.
Any other comments?