Skip to content

Use React Suspense when rendering async modals#28386

Merged
t3chguy merged 6 commits into
developfrom
t3chguy/async-wrapper
Nov 12, 2024
Merged

Use React Suspense when rendering async modals#28386
t3chguy merged 6 commits into
developfrom
t3chguy/async-wrapper

Conversation

@t3chguy
Copy link
Copy Markdown
Member

@t3chguy t3chguy commented Nov 5, 2024

No description provided.

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's React Suspense? Why is it good, and what are the practical implications of it?

@t3chguy
Copy link
Copy Markdown
Member Author

t3chguy commented Nov 6, 2024

What's React Suspense?

https://react.dev/reference/react/Suspense

Why is it good

Enables Concurrent Mode https://react.dev/blog/2022/03/29/react-v18#what-is-concurrent-react

and what are the practical implications of it?

  1. Aligning with the React way to do things
  2. Fulfilling a customer contract
  3. Speed

@t3chguy t3chguy requested a review from richvdh November 6, 2024 16:23
@richvdh
Copy link
Copy Markdown
Member

richvdh commented Nov 7, 2024

I'm going to wait for review from a member of the web team before looking at this. If the web team is happy, I'll check the crypto bits.

Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Comment thread src/Modal.tsx Outdated
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.

3 participants