search

microdotblog / microblog-react

MIT License
29 stars 3 forks source link

Double spinners #46

Closed paulrobertlloyd closed 1 year ago

paulrobertlloyd commented 1 year ago

It seems odd to see 2 spinners when refreshing content on a given screen. This is not how refreshing/loading indicators work across most apps, at least on iOS, probably for the reason that refreshing content shouldn’t hide that which is already displayed.

Screenshot of timeline screen
manton commented 1 year ago

Thanks. I think ideally the top spinner would only be used. Usually the top spinner just shows for a second and then the middle loading pane is for the real progress, so they should barely be on the screen at the same time. But good point.

vincentritter commented 1 year ago

Two things are happening here.

  1. We show a generic "Loading posts..." screen, in the Webview, when loading the content within. This is due to the fact that we cannot, as far as I know, keep a loading spinner going for the "pull to refresh" – it'll only trigger quickly.
  2. It also acts as a visual clue that you are, in fact, pulling to refresh.

Normally you shouldn't see both at the same time.

I'm going to close this one for now. But thank you for bringing it up.