-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[Hydration Errors] Replace HTML Diff tab #80092
Comments
Routing to @getsentry/product-owners-replays for triage ⏲️ |
@corbie1 thanks for catching that, I just put up #80581 to fix it! FYI we're looking for any feedback on whether that text diff is useful or not, and in what cases. As you can tell we haven't put a full amount of effort into this thing yet... one reason for that is there seems to be formatting changes that appear in the text diff which do not cause a hydration error, but do make the diff noisy. So if you have examples where this was helpful despite the implementation issues lmk! |
The HTML diff modal could be buggy with two known bugs that come up occasionally: (1) before & after were the same (2) one side was blank.
We propose replacing this tab with one of the following options:
The text was updated successfully, but these errors were encountered: