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

fixed issue #2632 #2665

Merged
merged 6 commits into from
Jul 6, 2020
Merged

fixed issue #2632 #2665

merged 6 commits into from
Jul 6, 2020

Conversation

kakugiki
Copy link
Contributor

@kakugiki kakugiki commented Feb 7, 2020

toContext2d: html2canvas.scale: (this.prop.pageSize.width * this.prop.pageSize.k) / Math.max(this.prop.src.clientWidth, this.prop.src.scrollWidth, this.prop.src.offsetWidth)

toContext2d: html2canvas.scale: (this.prop.pageSize.width * this.prop.pageSize.k) / Math.max(this.prop.src.clientWidth, this.prop.src.scrollWidth, this.prop.src.offsetWidth)
@HackbrettXXX
Copy link
Collaborator

Could you attach a small test case so I can test it?

@kakugiki
Copy link
Contributor Author

I'll send you the link when I get a chance. But I just figured out it should be easier and better to just set the scale to 1.

this is better than the math and able to keep the original scale/size
@kakugiki
Copy link
Contributor Author

https://kakugiki.github.io/jspdf_test/
tried zoom level at 500% and result is the same. Thanks.

@kakugiki
Copy link
Contributor Author

kakugiki commented Jul 5, 2020

@HackbrettXXX it's been a while, did you get a chance to look into this. This may also help to solve issue #2766.

@HackbrettXXX
Copy link
Collaborator

Looks good.

@HackbrettXXX HackbrettXXX merged commit f17e926 into parallax:master Jul 6, 2020
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.

None yet

3 participants