Skip to content

[pull] master from facebook:master#334

Merged
pull[bot] merged 2 commits intoAlan-love:masterfrom
facebook:master
Feb 22, 2021
Merged

[pull] master from facebook:master#334
pull[bot] merged 2 commits intoAlan-love:masterfrom
facebook:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Feb 22, 2021

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

rickhanlonii and others added 2 commits February 22, 2021 12:56
* Restore inspect-element bridge optimizations

When the new Suspense cache was integrated (so that startTransition could be used) I removed a couple of optimizations between the backend and frontend that reduced bridge traffic when e.g. dehydrated paths were inspected for elements that had not rendered since previously inspected. This commit re-adds those optimizations as well as an additional test with a bug fix that I noticed while reading the backend code.

There are two remaining TODO items as of this commit:
- Make inspected element edits and deletes also use transition API
- Don't over-eagerly refresh the cache in our ping-for-updates handler

I will addres both in subsequent commits.

* Poll for update only refreshes cache when there's an update

* Added inline comment
@pull pull bot added the ⤵️ pull label Feb 22, 2021
@pull pull bot merged commit cfd8c1b into Alan-love:master Feb 22, 2021
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.

1 participant