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

Popover does not disappear with angular 1.3.x #401

Closed
tzar opened this issue Nov 10, 2014 · 4 comments
Closed

Popover does not disappear with angular 1.3.x #401

tzar opened this issue Nov 10, 2014 · 4 comments
Labels

Comments

@tzar
Copy link

tzar commented Nov 10, 2014

From the docs, $animate.removeClass now has a different syntax:

removeClass(element, className, [options]); (returns promise)

textAngular passes a done callback which is never called, so the popup is not removed. To correct this, some modification like:

$animate.removeClass(scope.displayElements.popover, 'in').then(function(){

Would be needed, but I am not familiar with the testing framework used, and do not know how to make this also work for angular 1.2 cleanly.

As a sidenote, the animation classes seem to be missing from the copied CSS when running without bootstrap.

Cheers

@SimeonC SimeonC added the bug label Nov 10, 2014
@SimeonC
Copy link
Collaborator

SimeonC commented Nov 10, 2014

Thanks for letting me know, I'll get onto it when I can - from memory I think all I need to do to make your code snippet 1.2 compatible is this: $q.when($animate.removeClass(scope.displayElements.popover, 'in')).then(function(){

Can you let me know which version of textAngular are you using please?

@tzar
Copy link
Author

tzar commented Nov 10, 2014

Thanks for the speedy reply! I'm running 1.3.0-pre12, with the CSS from the src/ directory as it's not placed into dist/

@SimeonC
Copy link
Collaborator

SimeonC commented Nov 10, 2014

OK, will be fixed later today when I push up the next lot of fixes.

I'm not entirely sure what to do with the css, I've left it in the src folder as it isn't minified and I don't want two copies of it floating around.

@tzar
Copy link
Author

tzar commented Nov 10, 2014

Thanks 😄

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

No branches or pull requests

2 participants