Refresh on release

[andrewda]

Currently pages refresh when the page is pulled down. This doesn't feel natural — it should refresh on release instead.

[housseindjirdeh]

Are you referring to all pages that use PullToRefresh like the events screen/repository screen/org screen?

[andrewda]

Yes, as far as I know this happens on all screens.

[lex111]

What does "refresh on release" mean?

[tautf]

@lex111 i think @andrewda means when you release the screen so put away your finger the refresh action should start, right?

[andrewda]

Yea. Almost all apps I know of refresh when you let go, not when you pull up. Example:

[lex111]

@andrewda thanks, now I understand what you, although it's unusual for me, or I'm just used to the fact that now 😄

[housseindjirdeh]

So all pull to refresh functionality uses the RefreshControl (either directly or through FlatList as a onRefresh prop). So on iOS - it works lovely in my opinion

However if I'm not mistaken, the last time I tried with Android it acted a little weirdly. It wasn't really pulling the list was it? It just shows the loading indicator on pull and that's it. @andrewda @lex111 are you noticing the same thing?

Attached an android tag to this ticket but let me know if you happened to also be referring to iOS @andrewda

[andrewda]

Wait @housseindjirdeh try on a profile or repository. That's where it acts strangely for me (on iOS).

[tautf]

Yeah when you pull down and don't leave you finger it refreshes the content and acts a bit strange