Skip to content

Include Modes in the component stack - #13240

Merged
gaearon merged 3 commits into
react:masterfrom
gaearon:sm
Jul 19, 2018
Merged

gaearon merged 3 commits into
react:masterfrom
gaearon:sm

Conversation

@gaearon

@gaearon gaearon commented Jul 19, 2018

Copy link
Copy Markdown
Contributor

Alternative to #13109.

Now that the component stack is always attached automatically, this seems like a low-effort way to know the mode without exposing more methods on the isomorphic frame module. This should allow us to parse the stack to determine whether we're in a strict root, and be helpful when debugging (e.g. "where does the strict/async mode start in my tree?").

@bvaughn bvaughn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I dig this change.

@gaearon
gaearon merged commit 3d3506d into react:master Jul 19, 2018
@gaearon gaearon mentioned this pull request Sep 5, 2018
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
* Add a test that StrictMode shows up in the component stack

The SSR test passes. The client one doesn't.

* Include Modes in component stack

* Update other tests to include modes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants