Making angular-loading-bar be aware of 3rd party libraries using XHR requests by listening for events.

[yagoferrer]

Added feature: Be able to fire start / end request events from non-angular 3d party libraries that also do XHR requests so that the loading bar it's also aware of these requests.

In case that you are using Angular in combination with non-angular 3d party libraries, you can now fire custom 'cfpLoadingBar:start' and 'cfpLoadingBar:done' request events.

[yagoferrer]

This is useful to work with angular.js + oboe.js. It can also be achieved by adding start/done methods. I personally prefer events.

[rbutera]

this would be really useful for use with the Parse.com javascript SDK!

[matias-sandell]

Yes please!

[jturmel]

This seems like a great addition, actually needing it right now.

[ajhodges]

+1

[cspeer]

Let me give this a little bump +1
Is there any integration that works with parse.com js sdk? I would love to see that :)

[mattbrunetti]

@chieffancypants Are you interested in supporting this feature, if I resolve the merge conflicts?

[faceleg]

I don't use a 3rd party lib that does HTTP requests, I don't think @chieffancypants does either (so the motivation to merge and support this is going to be very low)

Also there are no tests in this PR proving that either the new feature does not break current functionality or that the new feature works as expected

[mattbrunetti]

@faceleg No tests here, true. I can write the tests for the new feature. The existing tests would prove that the new feature doesn't break current functionality, wouldn't they?

I think it's a pretty simple change, so it wouldn't be much extra to support, and it would add a lot of flexibility to this already excellent package. You would be able to use it when the user is waiting for, not just HTTP requests by 3rd party libs, but Web Workers, or requests over WebSockets. I'm sure there are other possibilities too. But it's up to you guys. If you're not interested I can figure something else out.

@faceleg @chieffancypants If I resolve the merge conflicts and add some tests, are you interested in this feature enough that you would review and consider merging?

[faceleg]

@mattbrunetti I'm interested yes