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

[ARRDT] Known excessive render update #79

Open
alloy opened this issue Jan 5, 2022 · 0 comments
Open

[ARRDT] Known excessive render update #79

alloy opened this issue Jan 5, 2022 · 0 comments
Assignees
Labels
wontfix This will not be worked on

Comments

@alloy
Copy link
Member

alloy commented Jan 5, 2022

These are known and will NOT be fixed, as they would require substantial complexity changes and are automatically fixed by virtue of updating to React v18 (when that goes stable).

usePaginationFragment

The new variables and the loading status changes are already batched, but the new data coming from the useFragment hook comes in a separate update.

(NOTE: I did get rid of the needless addition of id: undefined, but it does not cause any additional renders, just debugging noise.)

React v17

usePaginationFragment-React-v17

React v18

usePaginationFragment-React-v18

@alloy alloy added the wontfix This will not be worked on label Jan 5, 2022
@alloy alloy self-assigned this Jan 5, 2022
@alloy alloy changed the title [ARRDT] Known excessive render updates [ARRDT] Known excessive render update Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant