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

Normalize the coordinates used in SVGGraphics._makeTilingPattern (issue 12996) #13498

Merged
merged 1 commit into from
Jun 6, 2021

Conversation

Snuffleupagus
Copy link
Collaborator

While this prevents the error which is currently thrown by the assert in the DOMSVGFactory.create method, the pattern still doesn't actually render (visibly). However, in the interest of getting rid of some open issues, this patch should make (some) sense and there's already other issues about patterns in the SVG-backend,

Given that, as clearly outlined in the FAQ, the SVG-backend is not officially supported and that there's currently no development of it; this is probably the most that is reasonable to do here.

Fixes #12996

…ssue 12996)

While this prevents the error which is currently thrown by the `assert` in the `DOMSVGFactory.create` method, the pattern still doesn't actually render (visibly). However, in the interest of getting rid of some open issues, this patch should make (some) sense and there's already other issues about patterns in the SVG-backend,

Given that, as clearly [outlined in the FAQ](https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#backends), the SVG-backend is *not* officially supported and that there's currently no development of it; this is probably the most that is reasonable to do here.
@timvandermeij
Copy link
Contributor

/botio-linux preview

@pdfjsbot
Copy link

pdfjsbot commented Jun 6, 2021

From: Bot.io (Linux m4)


Received

Command cmd_preview from @timvandermeij received. Current queue size: 0

Live output at: http://54.67.70.0:8877/19e2b19ee372110/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Jun 6, 2021

From: Bot.io (Linux m4)


Success

Full output at http://54.67.70.0:8877/19e2b19ee372110/output.txt

Total script time: 3.77 mins

Published

@timvandermeij timvandermeij merged commit 7d83439 into mozilla:master Jun 6, 2021
@timvandermeij
Copy link
Contributor

Thanks!

@Snuffleupagus Snuffleupagus deleted the issue-12996 branch June 6, 2021 12:54
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.

SVG: "Invalid SVG dimensions"
3 participants