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

Support concurrent rendering in React 18 #127

Open
mixvar opened this issue Sep 2, 2022 · 0 comments
Open

Support concurrent rendering in React 18 #127

mixvar opened this issue Sep 2, 2022 · 0 comments
Assignees
Labels
perf performance optimization
Projects

Comments

@mixvar
Copy link
Collaborator

mixvar commented Sep 2, 2022

  • use useSyncExternalStore in internal useSubscription hook (use provided shim so that react peer dependency does not need to be bumped)
  • internal useSubscription hook should return value for use in affected hook/component (rather than accessing atom value directly)
  • remove FieldDependenciesAtom and extend FieldStateAtom to contain values for branch errors and branch isValidating
  • Atom.fuse may need to be extend with optional equality function so that branch erros / branch isValidating atoms can be created without extra re-renders
@mixvar mixvar added the perf performance optimization label Sep 2, 2022
@mixvar mixvar added this to To do in FormTS via automation Sep 2, 2022
@mixvar mixvar moved this from To do to In progress in FormTS Nov 5, 2022
@mixvar mixvar self-assigned this Nov 5, 2022
@mixvar mixvar moved this from In progress to To do in FormTS Feb 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
perf performance optimization
Projects
FormTS
To do
Development

No branches or pull requests

1 participant