nekomangaorg / Neko

Unofficial MangaDex Reader for Android 7+
Apache License 2.0
2.15k stars 112 forks source link

Download ahead causes page loading stutters on downloaded chapters. #1733

Closed CaidenBlock closed 3 months ago

CaidenBlock commented 4 months ago

Steps to reproduce

Enable download ahead

Download 2 consecutive chapters of any long strip that has a decent page count (count only makes it more noticeable)

Scroll between pages.

Expected behavior

Downloading new chapters in background shouldn't cause page loading stutters (like in 2.15.0)

Actual behavior

Stutters when scrolling between pages of already downloaded chapters

https://github.com/nekomangaorg/Neko/assets/34176510/b1785cec-9fc2-411b-a047-bca67ab77082

Crash logs

No response

Neko version

2.16.2

Android version

14

Device

Google Pixel 7 Pro

Other details

Repeatable across a fresh install on a Galaxy S22 (also A14), only changes from default are split pages (to make it more noticeable), and download ahead (set to 5)

Neither device does this with the same settings (as far as I can tell) on 2.15.0

I have a feeling this is caused by upstream, but I don't currently use Mihon and didn't see this reported there. ¯⁠\⁠⁠(⁠ツ⁠)⁠⁠/⁠¯

edit : did some testing in Mihon and did not experience this issue

Acknowledgements

CaidenBlock commented 4 months ago

Less noticeable in the clip I had chosen, but it stutters at the same time the page number on bottom changes, scrolling either up or down.

Here's an example where I scroll less wildly.

https://github.com/nekomangaorg/Neko/assets/34176510/5185969b-7a2d-48ab-9ca3-93ed5a0e5894

CaidenBlock commented 4 months ago

Expected (2.15.0) behavior:

https://github.com/nekomangaorg/Neko/assets/34176510/77684f1a-c911-426d-8c7a-22a7a72baf4f

nonproto commented 3 months ago

Still an issue it seems