Skip to content

Conversation

@narinluangrath
Copy link
Contributor

Fixes #157

@narinluangrath narinluangrath requested a review from gksander July 14, 2022 18:21
Copy link
Contributor

@gksander gksander left a comment

Choose a reason for hiding this comment

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

LGTM. What do you think about moving the <Highlight .../> markup into a constant so that we can dedup the example code, and just have

render(Content, document.getElementById('root'));

// const root = createRoot(document.getElementById('root'));
// root.render(Content);

I'm not married to that, but figured I'd offer it up.

@narinluangrath
Copy link
Contributor Author

@gksander Yeah that sounds good to me! I was thinking about doing that anyway.

Copy link
Contributor

@gksander gksander left a comment

Choose a reason for hiding this comment

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

:shipit:

@narinluangrath narinluangrath merged commit 9cb3d50 into master Jul 15, 2022
@narinluangrath narinluangrath deleted the docs/readme-react-18 branch July 15, 2022 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The first example from the documentation doesn't work

4 participants