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

Sync scheduling by default, with an async opt-in #11771

Merged
merged 1 commit into from
Jan 9, 2018

Conversation

acdlite
Copy link
Collaborator

@acdlite acdlite commented Dec 4, 2017

Removes the useSyncScheduling option from the HostConfig, since it's no longer needed. Instead of globally flipping between sync and async, our strategy will be to opt-in specific trees and subtrees.

Copy link
Collaborator

@sophiebits sophiebits left a comment

Choose a reason for hiding this comment

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

So there's no way to enable async for a whole page now?

@acdlite
Copy link
Collaborator Author

acdlite commented Dec 4, 2017

So there's no way to enable async for a whole page now?

We could do that by branching here:

https://github.com/acdlite/react/blob/78c69762e6f4e8ff158015e2d8d2426d7b1922db/packages/react-dom/src/client/ReactDOM.js#L1030

Is that something you think we'll want, for debugging purposes?

Removes the `useSyncScheduling` option from the HostConfig, since it's
no longer needed. Instead of globally flipping between sync and async,
our strategy will be to opt-in specific trees and subtrees.
@sophiebits
Copy link
Collaborator

Maybe. Not a huge deal.

Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

ok

@acdlite acdlite merged commit 13c5e2b into facebook:master Jan 9, 2018
ManasJayanth pushed a commit to ManasJayanth/react that referenced this pull request Jan 12, 2018
Removes the `useSyncScheduling` option from the HostConfig, since it's
no longer needed. Instead of globally flipping between sync and async,
our strategy will be to opt-in specific trees and subtrees.
chentsulin added a commit to chentsulin/Making-a-custom-React-renderer that referenced this pull request Jan 18, 2018
Removes the useSyncScheduling option from the HostConfig, since it's no longer needed. See facebook/react#11771
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.

4 participants