search

libreliodev / android

Android Customizable Reader - Librelio Digital Publishing Suite
www.librelio.com
226 stars 130 forks source link

Sometimes, covers view remains blank for a long time #302

Closed libreliodev closed 9 years ago

libreliodev commented 9 years ago

Subissue of #285.

intrications commented 9 years ago

Which covers are these? The large ones or small ones?

The large ones are downloading the newsstand images so will be slower to download. I guess the slower ones may be delayed by these bigger downloads, although the app should cancel downloads if the covers are scrolled passed before the download finishes.

intrications commented 9 years ago

The pop up dialogs are always using the newsstand images because they display bigger. Maybe this isn't required on a phone sized device.

libreliodev commented 9 years ago

@intrications

Which covers are these? The large ones or small ones?

See screenshot below

The large ones are downloading the newsstand images so will be slower to download. I guess the slower ones may be delayed by these bigger downloads, although the app should cancel downloads if the covers are scrolled passed before the download finishes.

Can't we display the covers view with generic covers while real covers are being downloaded? That's what we do on iOS.

The pop up dialogs are always using the newsstand images because they display bigger. Maybe this isn't required on a phone sized device.

Maybe, but I don't think this specific point is urgent.

intrications commented 9 years ago

@libreliodev

Ah, sorry. Misunderstood. I thought you meant the actual cover images were downloading slowly and they were appearing blank until the image downloads. I realise the correct problem now.

This issue is very puzzling to me but trying to debug it now.

intrications commented 9 years ago

@libreliodev

This is possibly fixed in the latest code. Haven't been able to test it enough yet but I did find a bug.

libreliodev commented 9 years ago

@intrications I have tested it, and noticed that there is no longer a blank screen when opening the covers view. However, when rotating the device, the blank screen appears and there does not seem to be a way to get out of it. Even when rotating back to the original orientation, the screen remains blank.

intrications commented 9 years ago

Hopefully finally all fixed.

libreliodev commented 9 years ago

This does seem to be fixed