edison
edison
> Any reason not to include test cases for the change to `hydration.ts`? It seems that `jsdom's innerHTML` does not normalize `\r\n` into `\n`. I did not figure out a...
> > > Any reason not to include test cases for the change to `hydration.ts`? > > > > > > It seems that `jsdom's innerHTML` does not normalize `\r\n`...
Thank you for your contribution. Maybe we should review https://github.com/vuejs/core/pull/10977 first. If the author is no longer active, we can consider creating a new PR.
`v-if` and `v-else` automatically generate a key. By adding `:key="1"` to v-if, both `v-if` and `v-else` end up with the same key of 1, which affects the patch logic.
/ecosystem-ci run
/ecosystem-ci run
I don’t think this is a complex feature, and perhaps there’s no need to go through an RFC for discussion. I prefer `Option 2` and made a PR, see #12868.
LGTM Similar PR https://github.com/vuejs/core/pull/10826
Thanks for your PR, it looks great. I think it should add some runtime tests in `dom/template.spec.ts` to verify these changes work as expected.
You can enable [VUE_PROD_HYDRATION_MISMATCH_DETAILS](https://vuejs.org/api/compile-time-flags.html#VUE_PROD_HYDRATION_MISMATCH_DETAILS) to display details in the production build.