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

Change position of loading bar #1001

Closed
bernardoadc opened this issue Jan 2, 2018 · 7 comments
Closed

Change position of loading bar #1001

bernardoadc opened this issue Jan 2, 2018 · 7 comments

Comments

@bernardoadc
Copy link

Specifically, I think it would be nice in the top of the page.

If not, then at least it should be corrected to be visible always, because right now it just stays below the visible screen. I can only see it if I scroll the page. But still, it makes more sense to me if it were on top.

@jung-kim
Copy link
Collaborator

jung-kim commented Jan 3, 2018

You mean the page load bar? I do agree that it's position is unreliable. If we were to do something like youtube esque loading bar would that be sufficient unless you have any other suggestions.

@bernardoadc
Copy link
Author

bernardoadc commented Jan 3, 2018 via email

@jung-kim
Copy link
Collaborator

jung-kim commented Jan 3, 2018

Sorry I meant, "youtube-esque"

Which is something like this: https://codepen.io/ozzysong/pen/vlIhx

@bernardoadc
Copy link
Author

bernardoadc commented Jan 3, 2018 via email

@jung-kim
Copy link
Collaborator

jung-kim commented Jan 7, 2018

Now I'm thinking maybe we should do for all api calls....

@bernardoadc
Copy link
Author

yes, it would be great! I've already caught myself waiting for something to happen and not sure it is doing anything.

It could be the same bar or a simple spinner (infinite/progress unknown)

@jung-kim
Copy link
Collaborator

jung-kim commented Jan 8, 2018

Yeah I think I would agree, it is being worked on at #1003.

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

No branches or pull requests

2 participants