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

Components Inside an Each Added to View Tree #428

Merged
merged 2 commits into from
Aug 1, 2015

Conversation

atsjj
Copy link
Contributor

@atsjj atsjj commented Jul 17, 2015

This should address #365, where components inside an {{#each}} are not
inspected.

Feedback on this is greatly appreciated!

This should address emberjs#365, where components inside an {{#each}} are not
inspected.
@atsjj atsjj changed the title Components Inside an Each Added to View Tree [WIP] Components Inside an Each Added to View Tree Jul 17, 2015
@atsjj
Copy link
Contributor Author

atsjj commented Jul 17, 2015

So, I am getting some odd behavior while inspecting and using my app at the same time. I decided to throw a [WIP] tag in the PR, because this is no where near ready to merge.

@ef4
Copy link
Contributor

ef4 commented Jul 29, 2015

This sees to work fine, and the code looks good to me. What kind of problems were you seeing?

* @return {Array} children
*/
_childrenForNode(renderNode) {
if (!!renderNode.morphMap) {
Copy link
Member

Choose a reason for hiding this comment

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

This is causing the linting failure (and subsequently the CI failure). Changing to:

if (renderNode.morphMap) { }

Should fix.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, thanks!

@atsjj
Copy link
Contributor Author

atsjj commented Aug 1, 2015

Sorry for the delay; busy week!

@ef4 the problem I'm having is when inspecting an object, it seems the bindings get all screwed up. I haven't had much time to get back into this, but try inspecting a route or controller and then trigger a transition to another route. That's where I was running into this problem.

@atsjj
Copy link
Contributor Author

atsjj commented Aug 1, 2015

@ef4 I can't reproduce the error any longer and everything is working as I'd expect.

@atsjj atsjj changed the title [WIP] Components Inside an Each Added to View Tree Components Inside an Each Added to View Tree Aug 1, 2015
ef4 added a commit that referenced this pull request Aug 1, 2015
Components Inside an Each Added to View Tree
@ef4 ef4 merged commit b2b62e2 into emberjs:master Aug 1, 2015
@Glavin001
Copy link

This is great! Thank you! When will this be published to Chrome extension?

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

Successfully merging this pull request may close these issues.

None yet

4 participants