Skip to content

Conversation

danbucholtz
Copy link
Contributor

Short description of what this resolves:

Changes proposed in this pull request:

Ionic Version: 1.x / 2.x

Fixes: #

…deprecated object, update docum

make the loader service return a promise instead of the deprecated object, update documents to show
proper usage
#3717

…deprecated object, update docum

make the loader service return a promise instead of the deprecated object, update documents to show
proper usage

#3717
@danbucholtz danbucholtz merged commit 0680930 into master May 10, 2016
@danbucholtz danbucholtz deleted the feature/loader-fix branch May 10, 2016 14:56
* $ionicLoading.show({
* template: 'Loading...'
* }).then(function(){
* console.log("The loading indicator is now displayed");
Copy link
Contributor

@dylanvdmerwe dylanvdmerwe May 10, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danbucholtz I really don't think the console.log outputs should be there.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is just the documentation on how to use it.

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

Successfully merging this pull request may close these issues.

2 participants