hypothesis / h

Annotate with anyone, anywhere.
https://hypothes.is/
BSD 2-Clause "Simplified" License
2.96k stars 427 forks source link

Add progress indicator while loading group members #9088

Open robertknight opened 5 days ago

robertknight commented 5 days ago

When group members are loaded, the user just sees a blank table. This is OK if the group members load quickly, but confusing if there is a delay for any reason.

See https://github.com/hypothesis/h/pull/9079 for context.

robertknight commented 3 days ago

The DataTable component has a loading prop that enables a built-in loading indicator. That is what we should use, although I think it needs some visual tweaks as the current loading indicator is quite large and it doesn't look good particularly if it very briefly flashes on screen while the content is loading.