Skip to content

Conversation

baiwusanyu-c
Copy link
Member

@baiwusanyu-c baiwusanyu-c commented Feb 22, 2023

close: #7775
close: #9229

@sxzz
Copy link
Member

sxzz commented Feb 22, 2023

/ecosystem-ci run

@vue-bot
Copy link
Contributor

vue-bot commented Feb 22, 2023

📝 Ran ecosystem CI: Open

suite result
nuxt ✅ success
pinia ✅ success
router ✅ success
test-utils ✅ success
vite-plugin-vue ✅ success
vitepress ✅ success
vue-macros ✅ success
vueuse ✅ success

@baiwusanyu-c baiwusanyu-c changed the title fix(hydration): Use escapeHTML when vnode is of text type fix(hydration): Use decodeHtml when vnode is of text type Feb 23, 2023
@vercel
Copy link

vercel bot commented Apr 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
sfc-playground ⬜️ Ignored (Inspect) Apr 6, 2023 0:44am

@github-actions
Copy link

github-actions bot commented Sep 15, 2023

Size Report

Bundles

File Size Gzip Brotli
runtime-dom.global.prod.js 100 kB (+413 B) 37.9 kB (+196 B) 34.1 kB (+142 B)
vue.global.prod.js 159 kB (+413 B) 57.8 kB (+175 B) 51.3 kB (+116 B)

Usages

Name Size Gzip Brotli
createApp 55.2 kB 21.2 kB 19.4 kB
createSSRApp 59.6 kB (+413 B) 23.1 kB (+188 B) 21.1 kB (+188 B)
defineCustomElement 59.9 kB 22.8 kB 20.7 kB
overall 68.9 kB 26.3 kB 23.9 kB

@edison1105
Copy link
Member

edison1105 commented Sep 16, 2023

take a closer look at ##7775.
style is a special case. see:

  • div
    image
  • style
    image
    I feel the best way is: to avoid escaping HTML for style at the compile phase or special treatment of style in the hydration phase

# Conflicts:
#	packages/runtime-core/src/hydration.ts
Copy link

netlify bot commented Jan 3, 2024

Deploy Preview for vue-next-template-explorer failed.

Name Link
🔨 Latest commit 7be8642
🔍 Latest deploy log https://app.netlify.com/sites/vue-next-template-explorer/deploys/6594cf01a60bdd0008bc5bfe

Copy link

netlify bot commented Jan 3, 2024

Deploy Preview for vue-sfc-playground failed.

Name Link
🔨 Latest commit 7be8642
🔍 Latest deploy log https://app.netlify.com/sites/vue-sfc-playground/deploys/6594cf018bb3f50008f53386

# Conflicts:
#	packages/runtime-core/src/hydration.ts
@baiwusanyu-c baiwusanyu-c force-pushed the bwsy/fix/hydrationText branch from 7be8642 to 9e9e1bf Compare January 3, 2024 08:06
`\n - expected on client: ${JSON.stringify(vnode.children)}`,
)
;(node as Text).data = vnode.children as string
let dataContent = (node as Text).data.replace(/[\t\r\n\f ]+/g, '')
Copy link
Member

Choose a reason for hiding this comment

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

Only \r or \r\n would actually cause a mismatch. \t\f and blank will not

Copy link
Member Author

Choose a reason for hiding this comment

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

done

baiwusanyu-c and others added 3 commits September 5, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs Review
Development

Successfully merging this pull request may close these issues.

Hydration error with \r in text #23234 SSR: CSS variable with quotes causes [Vue warn]: Hydration text mismatch
6 participants