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

feat(jsx/dom): introduce react-dom/client APIs and React.version #2795

Merged
merged 6 commits into from
May 26, 2024

Conversation

usualoma
Copy link
Member

Fixes #2792

version

There seems to be a library that does conditional branching on the major version number of React.version.

This is like a UserAgent of a web browser (e.g. Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/125.0.0.0 Safari/537. 36), which is a string for compatibility and somewhat unpleasant to use, and we do not recommend its use, but if this improves compatibility, it should be added.

createRoot / hydrateRoot

Some options are not yet supported, but they can be rendered in the way recommended by React since version 18, as shown in the following diff.

usualoma/hono-react-compat-demo@3877355

The author should do the following, if applicable

  • Add tests
  • Run tests
  • bun run format:fix && bun run lint:fix to format the code

@usualoma
Copy link
Member Author

@yusukebe How about including this in v4.4?

Copy link
Member

@yusukebe yusukebe left a comment

Choose a reason for hiding this comment

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

I've left a comment. And can you update the exports field in jsr.json?

src/jsx/dom/client.ts Show resolved Hide resolved
@yusukebe
Copy link
Member

@yusukebe How about including this in v4.4?

Nice! Let's include this.

@yusukebe
Copy link
Member

@usualoma

One thing. Please update the exports field in the jsr.json!

@usualoma
Copy link
Member Author

I'm fixing it, just give me a minute.

@usualoma
Copy link
Member Author

Sorry I'm late, fixed. 2e0dbca

Copy link
Member

@yusukebe yusukebe left a comment

Choose a reason for hiding this comment

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

LGTM!

@yusukebe
Copy link
Member

@usualoma

Thank you! Now, I'll merge this into the next. The v4.4 will be released tomorrow!

@yusukebe yusukebe merged commit 593cb7a into honojs:next May 26, 2024
10 checks passed
@usualoma usualoma deleted the feat-jsx-dom-client branch May 26, 2024 20:29
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.

2 participants