feat(server): useProps context #1385
Open
+167
−21
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #755
This change allows things like this:
So components can get access to the props from the root component by using the newly introduced
useProps
context.This is a generalization of the PR that was originally opened to close this issue. I thought it might be convenient to have not just the route available, but the entire props. I'm opening this as a draft since I didn't write any documentation yet, and I'm concerned about performance implications.
@marvinhagemeister, would you be so kind to clarify what sort of performance problems this will introduce? I see https://stackoverflow.com/questions/75060633/react-context-performance-and-suggestions and they talk about when context changes, the components re-render. Since we're just rendering this all on the server and the context is only created once and never changes, maybe it doesn't matter.
Mildly related to #1339.