mihon icon indicating copy to clipboard operation
mihon copied to clipboard

Weird slowdowns/reloads when opening individual title

Open asdkant opened this issue 1 year ago • 7 comments

Steps to reproduce

(This is an intermittent issue)

  1. open the details of any particular title in my library, wither though browsing the library or through browsing the source

Expected behavior

The details should load immediately (save the app's inherent latency), and control should return to the user

Actual behavior

The title "loads" as if I were refreshing the details (the one triggered by a swipe down), but even if there are new chapters they are not loaded, only the cover is refreshed if it's blank (covers also go blank way too frequently too, and specifically for titles in my library).

Crash logs

No response

Mihon version

0.16.1

Android version

11 (Samsung OneUI 3.1)

Device

Galaxy Tab S5e

Other details

I usually trigger this looking for new chapters in the "latest" search of sources, so I have to wait a few seconds for this "forced reload", then reaload again to get the new chapter(s)

This happens with multiple extensions so it seems unlikely that it's extension-specific.

Acknowledgements

  • [X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
  • [X] I have written a short but informative title.
  • [X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
  • [X] I have gone through the FAQ and troubleshooting guide.
  • [X] I have updated the app to version 0.16.1.
  • [X] I have updated all installed extensions.
  • [X] I will fill out all of the requested information in this form.

asdkant avatar Jan 21 '24 16:01 asdkant

🤔 Would you be able to provide a screen recording of this happening? And are you sure it isn't just the request to your source that is taking the time?

MajorTanya avatar Jan 21 '24 16:01 MajorTanya

I'll check how to do screen recording, tho I'll have to wait a bit to have a specific title to use as an example.

I'm sure it's not about the extensions because the problem is not the title refresh taking a few seconds (that's expected), the problem is that there IS a refresh at all when I just open the individual title view for a title that is already in my library therefore all the info should be available locally (even if it's a bit out of date).

asdkant avatar Jan 21 '24 17:01 asdkant

I'm not entirely sure but I believe that might happen sometimes to check stuff like covers or stale caches. Can't speak on details as I don't know the codebase for it though

MajorTanya avatar Jan 21 '24 17:01 MajorTanya

I'm not entirely sure but I believe that might happen sometimes to check stuff like covers or stale caches. Can't speak on details as I don't know the codebase for it though

I imagined the reason was something along those lines. It'd be nice to have an option to force a longer TTL for that local info so this issue and having blank covers are not triggered (especially for titles in my library)

asdkant avatar Jan 21 '24 17:01 asdkant

The starting point of the recording is when I open the recording is when I open the view for the specific title from browsing MangaDex. it automatically tries to refresh, finds nothing new, and then I manually refresh (it's a bit quick but you should be able to see it)

https://github.com/mihonapp/mihon/assets/3915542/9bb63022-197a-4e4f-a458-8893dc074d36

asdkant avatar Jan 22 '24 01:01 asdkant

The starting point of the recording is when I open the recording is when I open the view for the specific title from browsing MangaDex. it automatically tries to refresh, finds nothing new, and then I manually refresh (it's a bit quick but you should be able to see it) mihon-title-refresh.mp4

Thanks a lot for the screen recording, this might help someone figure it out! Looks like whatever that initial loading is, it's related to the cover image being refreshed or something — I could see it flash quickly, probably getting replaced. But it's definitely not doing a chapter check, interesting 🤔 Thank you again for the demo, now let us hope someone can figure this out.

MajorTanya avatar Jan 22 '24 01:01 MajorTanya

Looks like whatever that initial loading is, it's related to the cover image being refreshed or something (...)

Something to note: I also observed this happening without the cover having to be reloaded and not being changed after the forced refresh (may still be the cover but not be evident in that case)

asdkant avatar Jan 22 '24 11:01 asdkant