Skip to content

Conversation

eKoopmans
Copy link
Contributor

  • scaleFactor (k) is now correctly applied when calculating link rectangle and textWithLink height
  • Also removed redundant declaration of jsPDFAPI.link, which was declared twice

Closes #638
Related issues/PRs: #170, #390

@eKoopmans
Copy link
Contributor Author

Here's a demo of the problem behaviour. Notice that in the first example, the link positions do not correspond with the text/rectangle locations.

Current behaviour: jsFiddle
Behaviour after PR: jsFiddle

@Flamenco
Copy link
Contributor

Flamenco commented Apr 13, 2017

Great fiddle, and thanks for the contribution! I will try to get these in over the next few days.

@eKoopmans
Copy link
Contributor Author

Thanks!

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.

2 participants