fix(runtime-dom): patch textContent on svg properly

test: improve case chore: improve code chore: improve code chore: improve
vuejs · Aug 11, 2021 · 4bd1be0 · 4bd1be0
1 parent 1c7f5d3
commit 4bd1be0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/packages/runtime-dom/__tests__/patchAttrs.spec.ts b/packages/runtime-dom/__tests__/patchAttrs.spec.ts
@@ -10,6 +10,13 @@ describe('runtime-dom: attrs patching', () => {
     expect(el.getAttributeNS(xlinkNS, 'href')).toBe(null)
   })
 
+  test('textContent attributes /w svg', () => {
+    const el = document.createElementNS('http://www.w3.org/2000/svg', 'use')
+    patchProp(el, 'textContent', null, 'foo', true)
+    expect(el.attributes.length).toBe(0)
+    expect(el.innerHTML).toBe('foo')
+  })
+
   test('boolean attributes', () => {
     const el = document.createElement('input')
     patchProp(el, 'readonly', null, true)

diff --git a/packages/runtime-dom/src/patchProp.ts b/packages/runtime-dom/src/patchProp.ts
@@ -69,7 +69,7 @@ function shouldSetAsProp(
   if (isSVG) {
     // most keys must be set as attribute on svg elements to work
     // ...except innerHTML
-    if (key === 'innerHTML') {
+    if (key === 'innerHTML' || key === 'textContent') {
       return true
     }
     // or native onclick with function values