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

Our spinners are ugly. #15450

Closed
ara4n opened this issue Oct 13, 2020 · 3 comments · Fixed by matrix-org/matrix-react-sdk#5680
Closed

Our spinners are ugly. #15450

ara4n opened this issue Oct 13, 2020 · 3 comments · Fixed by matrix-org/matrix-react-sdk#5680

Comments

@ara4n
Copy link
Member

ara4n commented Oct 13, 2020

Screenshot 2020-10-13 at 10 34 47

n.b. the white fringing on the dark theme.

@ara4n ara4n added the T-Defect label Oct 13, 2020
@t3chguy
Copy link
Member

t3chguy commented Oct 13, 2020

There's a new spinner in Labs. Design needs to send one they are happy with though.

@niquewoodhouse
Copy link

I'm a bit concerned about this in a couple of ways.

1. Using spinners at all for this kind of thing.

Can we consider other ways of indicating progress? Skeleton UIs for example. Spinners are known to add to a sense of frustration and slowness in products. Short example post https://www.lukew.com/ff/entry.asp?1797. The more other products people use have things like skeleton UIs, the more spinners will stick out and add to that frustration.

2. Using the brand for spinners everywhere

I don't think it's a good idea to use the brand whenever there's something that needs a spinner, if spinners are needed. Leaving aside things like it potentially cheapening the brand (which is a very subjective discussion), there are studies around how custom spinners make users blame your product, and not their device or wifi, for example. https://blog.mercury.io/the-psychology-of-waiting-loading-animations-and-facebook/

@rosacola
Copy link

rosacola commented Nov 15, 2020

@niquewoodhouse just a appropriate spinner for the dark theme would suffice, the current one makes me uneasy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants