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

fix(runtime): don't flag html tag with hydrate flag anymore #6103

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

christian-bromann
Copy link
Member

@christian-bromann christian-bromann commented Jan 17, 2025

What is the current behavior?

fixes #6091

GitHub Issue Number: N/A

What is the new behavior?

Leave the document element alone and avoid hydration errors in React.

Documentation

Does this introduce a breaking change?

  • Yes
  • No

Potentially yes but I have doubts it has any impact on todays users as most of them use a bundler to ship their applications.

Testing

n/a

Other information

This was originally introduced in ionic-team/ionic-framework#18914

@christian-bromann christian-bromann requested a review from a team as a code owner January 17, 2025 19:26
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.

feat: don't hydrate class name in html tag anymore
1 participant