History view is limited to 20 items for each item type

babybuddy / babybuddy-for-android

Android client for the Baby Buddy webapp

MIT License

16 stars 6 forks source link

History view is limited to 20 items for each item type #16

Closed MrApplejuice closed 1 year ago

MrApplejuice commented 1 year ago

This can be sort-of annoying when trying to look back a few days. Ideally, the view would be an "infinite scrolling"-type view.

Should be changed in an update.

MrApplejuice commented 1 year ago

Found issues with the history item display:

~Add "quick update" feature if new items have to be loaded from the server~
- Not really needed anymore now because we preload enough items. Also, I do not want to flood the server.
[x] Add "quick update" when items are deleted
[x] Add "quick update" when timers are saved/events are registered
[x] Limit "removed items" to a max-size!
[x] Number of fetched items can likely be increased somewhat

MrApplejuice commented 1 year ago

Feature merged to master!

MrApplejuice commented 1 year ago

Some more testing shows that the new update routine is actually a lot slower than the old one. Need to investigate a better way to implement this :-/

MrApplejuice commented 1 year ago

Okay... this was solved, I think. I deferred the update cycles of the UI to a background-coroutine. Not ideal, but fast enough. When the interface 2.0 is implemented, it is likely a lot better to use a RecyclerView for the entire history-view!

MrApplejuice commented 1 year ago

Closing this, but this still needs some testing more...