PdfRendererView icon indicating copy to clipboard operation
PdfRendererView copied to clipboard

PDF quality is bad, it's unreadable and swiping experience is not good

Open l2dev opened this issue 6 years ago • 13 comments

For some reason pdfs look quite blurry

l2dev avatar Sep 11 '18 09:09 l2dev

Quality problem is fixed in 1.0.5. The new version has just uploaded. I'll investigate swiping problem. thanks.

jidogoon avatar Sep 12 '18 08:09 jidogoon

Nice!

How should I proceed with adding a progress message because now there is a delay (white window) before pdf is fetched.

l2dev avatar Sep 12 '18 15:09 l2dev

Nice!

How should I proceed with adding a progress message because now there is a delay (white window) before pdf is fetched.

I've updated README. You can set statusListener to show preparing status.

pdfView.statusListener = object: PdfRendererView.StatusCallBack { override fun onDownloadStart() { loading.visibility = View.VISIBLE } override fun onDownloadSuccess() { // downloading progress is not supported yet. will be added soon. loading.visibility = View.GONE } override fun onError(error: Throwable) { loading.visibility = View.GONE } }

jidogoon avatar Sep 12 '18 15:09 jidogoon

Nice! Thanks for ur support

On Wed, 12 Sep 2018 at 17:39, jidogoon [email protected] wrote:

Nice!

How should I proceed with adding a progress message because now there is a delay (white window) before pdf is fetched.

I've updated README https://github.com/jidogoon/PdfRendererView/blob/master/README.md. You can set statusListener to show preparing status.

pdfView.statusListener = object: PdfRendererView.StatusCallBack { override fun onDownloadStart() { loading.visibility = View.VISIBLE } override fun onDownloadSuccess() { // downloading progress is not supported yet. will be added soon. loading.visibility = View.GONE } override fun onError(error: Throwable) { loading.visibility = View.GONE } }

l2dev avatar Sep 12 '18 15:09 l2dev

Also what does it mean

/// downloading progress is not supported yet. will be added soon.

As I see after downloading is finished event is triggered. Do you mean that reporting status in terms of live percentage is not available?

l2dev avatar Sep 13 '18 07:09 l2dev

Also what does it mean

/// downloading progress is not supported yet. will be added soon.

As I see after downloading is finished event is triggered. Do you mean that reporting status in terms of live percentage is not available?

Yes. That's what I mean. I just released new version and updated README.

jidogoon avatar Sep 13 '18 08:09 jidogoon

Great!

I also noticed more problems:

I have a multiple pdf with many files. If I zoom into one of these pdf then I can not scroll down anymore on remaining PDF's.

Also in the general overview when I keep scrolling down through a file with few pages the whole scrolling animation seems lagging.

Maybe you should try opening a pdf with google docs viewer and compare it against yours, would be easier to find these problems ?

l2dev avatar Sep 13 '18 12:09 l2dev

Great!

I also noticed more problems:

I have a multiple pdf with many files. If I zoom into one of these pdf then I can not scroll down anymore on remaining PDF's.

Also in the general overview when I keep scrolling down through a file with few pages the whole scrolling animation seems lagging.

Maybe you should try opening a pdf with google docs viewer and compare it against yours, would be easier to find these problems ?

As you said, I have to improve behavior on swipe, scrolling and zoom. So I already made a feature using google docs viewer. It is now working only for Kitkat. As you suggested, I'll add option using google docs viewer even when It's not kitkat. I'll notify you when the option added.

jidogoon avatar Sep 13 '18 14:09 jidogoon

Great! I also noticed more problems: I have a multiple pdf with many files. If I zoom into one of these pdf then I can not scroll down anymore on remaining PDF's. Also in the general overview when I keep scrolling down through a file with few pages the whole scrolling animation seems lagging. Maybe you should try opening a pdf with google docs viewer and compare it against yours, would be easier to find these problems ?

As you said, I have to improve behavior on swipe, scrolling and zoom. So I already made a feature using google docs viewer. It is now working only for Kitkat. As you suggested, I'll add option using google docs viewer even when It's not kitkat. I'll notify you when the option added.

would be really nice, thanks!

l2dev avatar Sep 13 '18 14:09 l2dev

@jidogoon when can you add this flag for loading via google docs viewer?

l2dev avatar Sep 20 '18 12:09 l2dev

@jidogoon

There is a problem with google doc viewer (it's not fault of your implementation) where sometimes document is not shown and there is a blank screen. So I predict that your library will become more and more popular.

l2dev avatar Sep 21 '18 11:09 l2dev

@eurbon I've had some problems in development env. so I couldn't do something for new version. I just released new version. v1.0.7.

jidogoon avatar Oct 04 '18 03:10 jidogoon

@jidogoon getting error: duplicate value for resource 'attr/showDivider' with config ''. Message{kind=ERROR, text=error: duplicate value for resource 'attr/showDivider' with config ''., sources=[/Users/eimaurbon/.gradle/caches/transforms-1/files-1.1/appcompat-v7-28.0.0.aar/64cba8d69b0a6af81138ccd3ee31c5b8/res/values/values.xml:1304:5-69], original message=, tool name=Optional.of(AAPT)}

l2dev avatar Oct 04 '18 11:10 l2dev