Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

PdfRenderer instead of MuPDF #20

Closed
ibroom opened this issue Jun 29, 2016 · 3 comments
Closed

PdfRenderer instead of MuPDF #20

ibroom opened this issue Jun 29, 2016 · 3 comments

Comments

@ibroom
Copy link

ibroom commented Jun 29, 2016

Great plugin!

Is there any reason why you haven't used the Android PdfRenderer library such as demonstrated at https://github.com/googlesamples/android-PdfRendererBasic instead of depending on a 3rd party app that must be installed by users on Android?

I understand this library is only available on Android 5+ but it removes an external app dependency and makes it similar to iOS.

@regnete
Copy link
Member

regnete commented Dec 21, 2016

Could be integrated as an optional strategy for android

@matthewcts
Copy link

this is a great plugin, and i would like to ask if already have scheduled to integrated this new library (android pdfRenderer library) and make it more perfect.

@regnete
Copy link
Member

regnete commented Mar 2, 2017

We definitly won't integrate it within the next 12 months, but a contribution would be happlily accepted.

The new library renders an image for each page in the pdf. hyperlinks are not supported. so we would loose the linking feature which is not acceptable for all our use cases.

due to this reason it must be integrated as an additional/optional strategy for android which could be activated via an additonal option (https://github.com/sitewaerts/cordova-plugin-document-viewer#options).

html, css and javascript for displaying the rendered images could be reused from the windows part of this plugin (https://github.com/sitewaerts/cordova-plugin-document-viewer/tree/master/src/windows). we also have code to extract and display outline/toc there, and event pinch/zoom is supported. The available code should be splitted in a platfrom independend and a platform dependend part. After that the android specific code could be added.

So if anyone out here is able and willing to implement this feature: go ahead!

@regnete regnete closed this as completed Aug 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants