jsPDF icon indicating copy to clipboard operation
jsPDF copied to clipboard

Links breaks when use addPage with another size

Open MarceloZapatta opened this issue 4 years ago • 5 comments

Hi!

I'm facing an issue, links breaks when .addPage() with another format than the initial size, apparently the position of the link became incorrect.

Reproduce:

  • Start new doc with an specific size
  • Put some link anywhere in the first page
  • addPage() with an different size
  • The link inserted in the first page doesn't respect anymore the position passed.

Notice that this occurs with textWithLink and also link(). If the addPage() has the same size of initial format links work's just fine. I made an fiddle to help:

Works: https://jsfiddle.net/MarceloZapatta/don4hqtj/

Breaks: https://jsfiddle.net/fL403sja/2/

MarceloZapatta avatar Jul 06 '20 12:07 MarceloZapatta

Yes, that's a bug. The links are written when the pages are finalized. Here the getHorizontal/VerticalCoordinateString functions are no longer valid (they use the dimensions of the last active page). The final bounds need to be stored when the link function is executed.

HackbrettXXX avatar Jul 06 '20 12:07 HackbrettXXX

Hey I would love to work on this issue. Could you please provide some more information on it?

rahil1304 avatar Oct 03 '20 12:10 rahil1304

Thanks! What additional information do you need exactly?

HackbrettXXX avatar Oct 05 '20 09:10 HackbrettXXX

@rahil1304 I just realized that @MarceloZapatta already made a PR, sorry about that.

HackbrettXXX avatar Oct 05 '20 10:10 HackbrettXXX

@HackbrettXXX Hi this issue has been resolved since 3 years can you close it ?

Setha77 avatar Jan 02 '24 15:01 Setha77