$Service is now deprecated. Use $uibService instead.

[declanelcocks]

I've updated to 0.14.0 to make use of the fixed tooltips and a few other things and I now have the warning listed in the comment. Are there some ongoing issues regarding migrating to 0.14.0? For example, I have no instances of $tooltip used within my app, but I still get the warning to use $uibTooltip and to use uib-tooltip-html-popup. Similarly, after replacing all $modal instances with $uibModal, I still receive the warnings. I did also try updating the directives on the page to uib-tooltip-... as well.

[maxklenk]

Even if your are not using the $tooltip service, the message tells you to replace all tooltip classes and directives like:

<a tooltip="Hey">  => <a uib-tooltip="Hey">

[Foxandxss]

There could be some edge cases where you see a deprecation warning when you shouldn't.

I tracked lot of them and I wasn't able to get more.

Check the migration guide (link on readme) and see if you're missing something.

[declanelcocks]

I found the issue causing the $tooltip error, but it was actually being caused by a popover instance which was misleading. I know that popovers are created using tooltip, but it was still a bit misleading. Is there any migration for tooltip-class? I see that in the source you are using uib-tooltip-classes but this doesn't seem to be working as tooltip-class did.

[Foxandxss]

@declanelcocks Can you create a plunker with the popover that is causing that issue for you?

[declanelcocks]

@Foxandxss Sorry for slacking on the replies, I had to move on to working on something else first as this was a minor issue in my project. I will try to play around with this before the weekend and let you know how it goes. I believe after updating the demo page, it's a bit easier to understand the migration so it may just be a simple error on my end.

[Dilatorily]

@Foxandxss I was able to reproduce a misleading deprecation warning with the popover class.

Here is a plunkr of the issue.

[declanelcocks]

@Dilatorily Correct, I receive warnings for this as well but the warning should be a bit more explicit as you find yourself looking through the source code for tooltip rather than popover

[Dilatorily]

@declanelcocks I agree. Other libraries use the Bootstrap's popover class, and they will create a false deprecation warning.

If the warnings could be clearer by having the right source, and by having the right use case (element, attribute, or class), it would have saved me a lot of headaches.

[Foxandxss]

@Dilatorily Angular will process all classes in the HTML and check if they are directives. Since tooltip is a directive, Angular will wake up the tooltip directive, also wake up all of his injections and bam, $tooltip will cry.

I spent at least 5 hours figuring it out and we had to do some workarounds to avoid that.

I think I can go one step further to make it better. Will investigate.

Also, I will investigate if I can make the warning a bit more useful.

[declanelcocks]

@Foxandxss It's a very picky improvement so it's not much of a big deal 😊

[Foxandxss]

It was a dead end at the end.

I am playing with logging the element and attributes for each deprecation. Way more verbose now, so I am not entirely sure if it is a good idea.

Will discuss it with the other guys.