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

Add tooltip to loading spinner #394

Closed
cassidyjames opened this issue Jul 11, 2017 · 3 comments · Fixed by #1919
Closed

Add tooltip to loading spinner #394

cassidyjames opened this issue Jul 11, 2017 · 3 comments · Fixed by #1919
Labels
Good First Issue Small, self-contained issue. Good for newcomers, and/or should be an easy fix. Priority: Low On the radar, but not the most urgent thing Status: Confirmed Verified by someone other than the reporter

Comments

@cassidyjames
Copy link
Contributor

cassidyjames commented Jul 11, 2017

It might not be clear what exactly is loading when the spinner in the headerbar is going. We could reveal some more information as a tooltip for those curious.


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@danirabbit danirabbit added Priority: Low On the radar, but not the most urgent thing Status: Confirmed Verified by someone other than the reporter labels Jul 11, 2017
@cassidyjames
Copy link
Contributor Author

All we have a concept of right now is "working" or not. Is it worth adding a single tooltip, or should we add multiple states (i.e. "Loading trending apps", "Checking for updates", etc.)?

@megatux
Copy link
Contributor

megatux commented Nov 30, 2018

Also the position of the spinner so close to the search input makes it more confused of its real meaning.
Probably [Needs Design] label?

@megatux
Copy link
Contributor

megatux commented Jan 28, 2019

"Loading trending apps", "Checking for updates", etc, sound good to me. Are these fetching processes made in serial or parallel ?

@davidmhewitt davidmhewitt added the Good First Issue Small, self-contained issue. Good for newcomers, and/or should be an easy fix. label Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Small, self-contained issue. Good for newcomers, and/or should be an easy fix. Priority: Low On the radar, but not the most urgent thing Status: Confirmed Verified by someone other than the reporter
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants