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

(IE only) Last element created by <template repeat> is unbound #187

Closed
morethanreal opened this issue Jun 20, 2013 · 4 comments
Closed

(IE only) Last element created by <template repeat> is unbound #187

morethanreal opened this issue Jun 20, 2013 · 4 comments
Assignees

Comments

@morethanreal
Copy link
Contributor

@sorvell
Gist: https://gist.github.com/morethanreal/5825813

This example creates two x-foos in a <template repeat>. On IE, the last x-foo is unbound. It seems like for the last element the insertedCallback which cancels the unbind was fired after asyncUnbindAll has already taken effect.

It's only a problem when you use <template repeat> to make custom elements.

@sorvell
Copy link
Contributor

sorvell commented Jun 21, 2013

We need to verify if the insertedCallback is fired too late or not fired at all.

On IE, the Mutation Observers polyfill uses mutation events. It's my understanding that they are synchronous so it seems unlikely that there's an ordering issue and more likely that the event is not firing at all.

@morethanreal
Copy link
Contributor Author

I verified that it is firing by adding an inserted callback on x-foo. It
comes after the unbindAll call.
On Jun 21, 2013 4:49 PM, "Steve Orvell" notifications@github.com wrote:

We need to verify if the insertedCallback is fired too late or not fired
at all.

On IE, the Mutation Observers polyfill uses mutation events. It's my
understanding that they are synchronous so it seems unlikely that there's
an ordering issue and more likely that the event is not firing at all.


Reply to this email directly or view it on GitHubhttps://github.com//issues/187#issuecomment-19846652
.

@ghost ghost assigned morethanreal Sep 25, 2013
@sorvell
Copy link
Contributor

sorvell commented Sep 25, 2013

@morethanreal Can you verify if this is still a problem? Thanks.

@morethanreal
Copy link
Contributor Author

Seems to be fixed.

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

No branches or pull requests

2 participants