search

ephemeraHQ / converse-app

https://converse.xyz
MIT License
39 stars 3 forks source link

Conversation Loading States #353

Closed alexrisch closed 2 weeks ago

alexrisch commented 1 month ago

When errors/loading is happening it can be helpful to inform the user in some way Right now loading and errors are displayed as a spinner at the top of the conversation list, but does not give insight into a problematic conversation

lourou commented 3 weeks ago

We could display the loading state next to the spinner. This would help the user understand what's going on. We could also show the state if the spinner has been loading for more than 10/15 seconds.

lourou commented 2 weeks ago

I have submitted a pull request with a proposed set of enhancements, screenshots attached: https://github.com/ephemeraHQ/converse-app/pull/561

Screenshots

Regular loading (below 15 seconds and without ongoing errors): IMG_1186

Reconnecting: IMG_1185

With long user name, shorten, as seen on iPhone 13 Mini (smallest screen width on iOS) IMG_1187

Waiting for network: Waiting

saulmc commented 2 weeks ago

TODO: Consider wording of "Reconnecting" (currently: weird state or cannot connect with XMTP network) and "Waiting for network" (currently: can we communicate with Converse backend?)

lourou commented 2 weeks ago

@saulmc, this is completed, merged, and the issue has been automatically closed. Let me know if any wording change is necessary.