Page MenuHomePhabricator

Loading state for notifications center table
Closed, DeclinedPublic

Assigned To
None
Authored By
cmadeo
Aug 13 2021, 3:39 PM
Referenced Files
F34597893: ActivityIndicator.png
Aug 16 2021, 10:59 PM
F34594823: Spinner.pdf
Aug 13 2021, 7:42 PM
F34594825: image.png
Aug 13 2021, 7:42 PM
F34594819: image.png
Aug 13 2021, 7:39 PM
F34594822: Spinner.pdf
Aug 13 2021, 7:39 PM

Description

Why are we doing this?

For contributors who have a large number of messages, we will need to be able to present a loading state as the table view is loaded at the end of each 'page' load.

Design
image.png (812×375 px, 49 KB)

Design details

  • Image should rotate (spin) until the new set of notifications is loaded
  • Spinner appears at the bottom of the loaded section of the table.
  • Utilize the theme's 'primary action' color (eg #36C for default) for the color of the spinner
Dependencies

https://phabricator.wikimedia.org/T288662

Event Timeline

@cmadeo Nice. I wonder if it might be wise to rely on the system's standard activity indicator here (image below) to begin with as it both already connotes a loading action to users across tons of apps on the platform and we've leveraged it as so as a table footer element in the article as a living document modal as the user paged in more edits.

Happy to go with the custom UI approach if you feel strongly about it though.

ActivityIndicator.png (200×200 px, 772 B)

From @JMinor : While loading new notifications, it’s fine to block the Notification Center screen, for as long as needed. (And there may be screens from Carolyn for this already.) We don’t need to worry about how long this might take, and we can also do this when adding a new language.

Tsevener added a subscriber: Tsevener.

Closing this one for now, it doesn't seem needed because of bulk import. Local paging is very fast.