MobileOrg / mobileorg

MobileOrg iPhone App
http://mobileorg.github.io
GNU General Public License v2.0
556 stars 69 forks source link

Implement pull to refresh behavior #270

Open webframp opened 4 years ago

webframp commented 4 years ago

This is a pretty standard behavior on iOS for forcing a refresh action. We currently only have the sync button on the main outlines view, but could make pull to refresh work from any screen for consistency.

dive commented 4 years ago

Just a note.

It is quite easy to implement but the problem is that we block UI during the refresh for now. It was done to protect the database from any changes during the synchronisation process.

So, we can implement a transparent synchronisation for sure and do not block UI at all (we have to support merging, etc., if a user changes something during the synchronisation then we have to merge these changes back when synchronisation is completed). But I think it will take some time and will introduce some breaking changes. Anyway, we have to take the current implementation into account.

webframp commented 4 years ago

Great details @dive

That’s for sure why it doesn’t currently exist, I wanted to have an issue for it since I’m trying to make it easier for contributors to add their thoughts or see what features have been discussed.