-
Notifications
You must be signed in to change notification settings - Fork 140
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
Validation Error on Default Scene Render #291
Comments
Thanks for the report. I think the correct fix might be to skip the parts of the render which fail if there are no paths, which may well be everything before For now, in user code, a workaround would be to always render something, e.g. a triangle of the background colour. |
Clearing the target to a default color was actually something, that I was hoping for, when first stumbling across this bug. That would seem like a reasonable solution to me. 👍 |
…518) * WIP: resurrect pico_svg and use it to load scenes. * Add viewbox and width/height support to pico_svg. * Accept more single-function transforms in pico_svg. * Add copyright header to pico_svg.rs * Remove vello_svg. * Clean up pico_svg lints. * Parse multiple transforms in attribute. * Add scaleX and scaleY to pico_svg. * Render fuchsia unit square on error, to work around #291. * Print Display rather than Debug for Err in svg.rs * Interpret combined viewBox and width/height correctly. * Fix lint. * Clean up std::result qualified names. * Mention new vello_svg repository in ## Integrations * Improve color parsing in pico_svg. * Address review comments.
Alternative to linebender#629 Fixes linebender#291
The following example tries to render an empty scene.
The code crashes at runtime with the following output:
The same happens, when rendering an empty scene to a
winit
window.The text was updated successfully, but these errors were encountered: