espresso3389 / pdfrx

pdfrx is yet another PDF viewer implementation that built on the top of pdfium. The plugin currently supports Android, iOS, Windows, macOS, Linux, and Web.
MIT License
60 stars 36 forks source link

Could not handle certain destination coordinates correctly #135

Closed AdityaHighspot closed 3 weeks ago

AdityaHighspot commented 3 weeks ago

Hey @espresso3389 I'm wondering why we are able to use linkWidgetBuilder for handling external URLs, but don't have support for internal URLs on it?

e.g. when you click on a link that's pointing to another page in the same document, it should also be handled inside the same function. PFA sample PDF with links on top for browsing through the pages. dripbox.pdf

espresso3389 commented 3 weeks ago

pdfrx does support in-PDF links but does not handle links in the document...

AdityaHighspot commented 3 weeks ago

Can you please tell me how to handle such links then?

espresso3389 commented 3 weeks ago

It is just discovered that the PDF file has broken links. The rectangles's top and bottom coordinates are stored in wrong order (at least not expected order). Anyway, we can easily detect/auto-fix such error so the newer code can handle them correctly.

espresso3389 commented 3 weeks ago

1.0.56 includes the fix.