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

Loading bar should only appear on noticeable latency #27

Closed
chieffancypants opened this issue Dec 12, 2013 · 6 comments
Closed

Loading bar should only appear on noticeable latency #27

chieffancypants opened this issue Dec 12, 2013 · 6 comments
Assignees
Milestone

Comments

@chieffancypants
Copy link
Owner

The bar should only appear when there is noticeable latency. Displaying the bar on every XHR request gets repetitive, and serves very little purpose on those quick ~20ms calls.

This change will likely break lots of tests

@ghost ghost assigned chieffancypants Dec 12, 2013
chieffancypants added a commit that referenced this issue Dec 12, 2013
chieffancypants added a commit that referenced this issue Dec 12, 2013
alexmingoia added a commit to alexmingoia/angular-loading-bar that referenced this issue Jan 3, 2014
@alexmingoia
Copy link
Contributor

fixed the configuration... will try and wrap my head around the tests and update them this weekend.

@luisrudge
Copy link

this isn't in bower yet, right?

@chieffancypants
Copy link
Owner Author

@luisrudge nope, not yet -- still working in a separate branch for now

@chieffancypants
Copy link
Owner Author

@alexmingoia you're cool if I merge your work into my branch right? I was gonna take a stab at the tests soon

@alexmingoia
Copy link
Contributor

@chieffancypants Yes, please do :)

chieffancypants added a commit that referenced this issue Jan 21, 2014
Added angular libs via bower so I don't have to do this again
chieffancypants added a commit that referenced this issue Jan 21, 2014
The extra $timout.flush() accounts for the latency threshold
@chieffancypants
Copy link
Owner Author

This one should be all set guys. Let me know if you encounter any weirdness.

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

No branches or pull requests

3 participants