Browse Source

Always use textContent instead of innerText

Philipp Bosch 4 năm trước cách đây
mục cha
commit
f70a71ef42
2 tập tin đã thay đổi với 4 bổ sung8 xóa
  1. 1 5
      src/directives/text.js
  2. 3 3
      test/text.spec.js

+ 1 - 5
src/directives/text.js

@@ -4,9 +4,5 @@ export function handleTextDirective(el, output, expression) {
         output = ''
     }
 
-    if (el instanceof HTMLElement) {
-        el.innerText = output
-    } else {
-        el.textContent = output
-    }
+    el.textContent = output
 }

+ 3 - 3
test/text.spec.js

@@ -14,7 +14,7 @@ test('x-text on init', async () => {
 
     Alpine.start()
 
-    await wait(() => { expect(document.querySelector('span').innerText).toEqual('bar') })
+    await wait(() => { expect(document.querySelector('span').textContent).toEqual('bar') })
 })
 
 test('x-text on triggered update', async () => {
@@ -28,11 +28,11 @@ test('x-text on triggered update', async () => {
 
     Alpine.start()
 
-    await wait(() => { expect(document.querySelector('span').innerText).toEqual('') })
+    await wait(() => { expect(document.querySelector('span').textContent).toEqual('') })
 
     document.querySelector('button').click()
 
-    await wait(() => { expect(document.querySelector('span').innerText).toEqual('bar') })
+    await wait(() => { expect(document.querySelector('span').textContent).toEqual('bar') })
 })
 
 test('x-text on SVG elements', async () => {