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.

Sign up for GitHub

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

Jump to bottom

added example of component mutating an already rendered property #1

Merged
merged 1 commit into from Dec 10, 2016
Merged

added example of component mutating an already rendered property #1

merged 1 commit into from Dec 10, 2016

Conversation

GavinJoyce
Copy link
Owner

@GavinJoyce GavinJoyce commented Dec 10, 2016 •
edited

screen shot 2016-12-10 at 12 37 01

@GavinJoyce GavinJoyce merged commit 58ddfb4 into master Dec 10, 2016
@GavinJoyce GavinJoyce deleted the gj/example-1 branch December 10, 2016 12:46
@GavinJoyce GavinJoyce mentioned this pull request Dec 11, 2016
Closed
@GavinJoyce
Copy link
Owner Author

Here's a nice tip from Kris Selden:

screen shot 2016-12-12 at 13 05 11

For this issue, here's how it helps:

  1. Put a breakpoint in assertNotRendered and note the key:

screen shot 2016-12-12 at 13 00 15

  1. Put a conditional breakpoint with the same key in didRender:

screen shot 2016-12-12 at 13 00 41

  1. Reload the app and you'll break at the original place where the value was first rendered:

screen shot 2016-12-12 at 13 01 14

  1. Navigate up the stack until to the currently executing opcode, you can determine the template from this information:

screen shot 2016-12-12 at 13 03 57

screen shot 2016-12-12 at 13 04 27

@GavinJoyce GavinJoyce mentioned this pull request Dec 12, 2016
Closed
@bstro
Copy link

bstro commented Mar 12, 2018

What do you mean by "navigate...to the currently executing opcode" ?

@GavinJoyce
Copy link
Owner Author

GavinJoyce commented Mar 12, 2018 •
edited

@bstro these are a little out of date, but should still help:

https://www.youtube.com/watch?v=vL8sCi1Bv6E&index=1&list=PLpAr6J-75N27wctNT70O0lubaGTPjwi1L

https://www.youtube.com/watch?v=l2nD1IxjHUI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@GavinJoyce @bstro