Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove delivery spinners entirely #79

Closed
quinthar opened this issue Aug 12, 2020 · 1 comment · Fixed by #112
Closed

Remove delivery spinners entirely #79

quinthar opened this issue Aug 12, 2020 · 1 comment · Fixed by #112
Assignees

Comments

@quinthar
Copy link
Contributor

let's not show the spinner after sending a message because it highlights how long it takes for the message to actually be delivered, whereas it would be nicer if it were presented as if instantaneous and only show that it wasn't delivered in the event there is some kind of error

@quinthar quinthar added this to the Alpha Launch milestone Aug 12, 2020
@quinthar
Copy link
Contributor Author

Incidentally, the spinners seem broken to me anyway; they never stop spinning, even though the message was delivered. So I suggest for now, the critical thing is to just remove the spinners entirely. I'll create a second issue to add a "queued" indicator for messages that can't be delivered (either due to network error, or being written while offline)

@quinthar quinthar changed the title Hide spinner when sending; only show on error Remove delivery spinners entirely Aug 12, 2020
@Jag96 Jag96 assigned Jag96 and unassigned AndrewGable Aug 12, 2020
rushatgabhane pushed a commit to rushatgabhane/exfy that referenced this issue May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants