This repository has been archived by the owner on May 29, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 6.7k
tooltip doesn't work with ng-animate and bootstrap 3.3.1 #2959
Comments
I noticed this too. Not sure what is going on but ngAnimate (I think) it is not liking objects with two keys in the ng-class of the template. You can work around it for the moment by adding: .run(function($templateCache) {
$templateCache.put("template/tooltip/tooltip-popup.html",
"<div class=\"tooltip fade {{placement}}\" ng-class=\"{ in: isOpen() }\">\n" +
" <div class=\"tooltip-arrow\"></div>\n" +
" <div class=\"tooltip-inner\" ng-bind=\"content\"></div>\n" +
"</div>\n" +
"");
}); Notice that I have moved "fade" from the ng-class object to class. (you can also remove ngAnimate or drop down to 1.2.6) |
+1 |
This is already fixed on master. |
Hi. No, it isn't. Take a look at my link. I compiled your latest commit and used it in the example |
Indeed, I forgot to push the fix for #2951 . This should be fixed now. |
OronNadiv
pushed a commit
to lanetix/bootstrap
that referenced
this issue
Nov 18, 2014
In Angular 1.3.1, when using tooltips with ngAnimate, this extra $digest call seems to break ngClass. This is an extra call to ngAnimate and it shouldn’t be called since the tooltip hasn’t been added to the DOM. I was unable to create a test case to test compatibility with ngAnimate. Fixes angular-ui#2951 Fixes angular-ui#2959
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
http://plnkr.co/edit/AWYJyku4YJGv7UBPrRha?p=preview
Works without ngAnimate only (try to change in script.js)
The text was updated successfully, but these errors were encountered: