Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

ngAnimate ngClass digest fixes #3717

Closed
wants to merge 2 commits into from
Closed

Conversation

matsko
Copy link
Contributor

@matsko matsko commented Aug 23, 2013

No description provided.


while(postDiquestQueue.length) {
try {
target.$eval(postDiquestQueue.shift());
Copy link
Contributor

Choose a reason for hiding this comment

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

this should not be $eval, we should just call the fn in the queue without $eval or binding it to any context

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK Done. Should I still keep a try {} / catch {} around it?

@IgorMinar
Copy link
Contributor

lgtm otherwise

@matsko
Copy link
Contributor Author

matsko commented Aug 23, 2013

@IgorMinar there seems to be still something broken with ngClass and $$postDigest... Don't merge yet...

@ghost ghost assigned mhevery Aug 23, 2013
@matsko matsko closed this Aug 23, 2013
@matsko
Copy link
Contributor Author

matsko commented Aug 23, 2013

Closing this so it doesn't accidentally get merged.

@mernen
Copy link
Contributor

mernen commented Aug 26, 2013

$$postDigest is a very useful feature though (e.g. #734). Are you still planning to include it in a future patch, and possibly as public API?

@matsko
Copy link
Contributor Author

matsko commented Aug 26, 2013

@mernen I agree, but if it has a double dollar then it's hidden for the time being. What do you think @IgorMinar about making it public down the road?

@IgorMinar
Copy link
Contributor

Maybe. We need to learn how well it works before we commit to maintaining it as a public api.

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

Successfully merging this pull request may close these issues.

4 participants