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

Bug in edge, generating too many components #105

Closed
nmingneau opened this Issue Nov 14, 2018 · 5 comments

Comments

Projects
None yet
5 participants
@nmingneau
Copy link

nmingneau commented Nov 14, 2018

https://codepen.io/nmingneau/pen/PxWPLm?editors=1010

This codepen generates a different result in edge compared to chrome, firefox and safari.
Chrome shows one test string, while edge shows two.

@chasenlehara

This comment has been minimized.

Copy link
Member

chasenlehara commented Nov 15, 2018

Reduced down a little bit more: https://codepen.io/chasenlehara/pen/rQmeEj

@nmingneau

This comment has been minimized.

Copy link

nmingneau commented Nov 23, 2018

@phillipskevin Any news on this issue?

@justinbmeyer

This comment has been minimized.

Copy link
Contributor

justinbmeyer commented Nov 23, 2018

I think we added it to this sprint

@justinbmeyer

This comment has been minimized.

Copy link
Contributor

justinbmeyer commented Nov 23, 2018

Why is in can-construct? Seems more likely to be can-component.

@justinbmeyer justinbmeyer transferred this issue from canjs/can-construct Nov 23, 2018

@justinbmeyer

This comment has been minimized.

Copy link
Contributor

justinbmeyer commented Nov 23, 2018

viewCallbacks.tag("edge-double-insert-inner", function(el, tagData){
  console.log("INNER");
});


document.getElementById("qunit-fixture").innerHTML = "<edge-double-insert-outer></edge-double-insert-outer>"

viewCallbacks.tag("edge-double-insert-outer", function(el, tagData){
   el.innerHTML = "<edge-boudle-insert-inner></edge-boudle-insert-inner>";
   viewCallbacks.tagCallback(el.firstChild, "edge-boudle-insert-inner", {});
});
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment