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

fix: Derive playground values instead of storing them in state #5067

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

cephalization
Copy link
Contributor

@cephalization cephalization commented Oct 17, 2024

This prevents stale variable inputs from being displayed on the playground.

TODO: fix render slowdown. This approach is slower, need to memoize some more things.

Performance profiling reveals that dropped frames while typing quickly is due to expensive over-re-rendering. We can fix this later or via react compiler in the future.

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Oct 17, 2024
@cephalization
Copy link
Contributor Author

Resolves #5060

@cephalization cephalization force-pushed the cephalization/derived-input-variables branch from 2d54301 to cae060b Compare October 17, 2024 20:58
@cephalization
Copy link
Contributor Author

To verify the fix, just enter the playground via any span and observe that the inputs tab is not pre-populated with incorrect variables

@cephalization cephalization merged commit d935d4b into main Oct 18, 2024
7 checks passed
@cephalization cephalization deleted the cephalization/derived-input-variables branch October 18, 2024 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:L This PR changes 100-499 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants