Merge pull request #437 from taddei/master

[Fix] added JSXText as a valid accessible child

Author: ljharb

__tests__/src/util/hasAccessibleChild-test.js

@@ -36,6 +36,15 @@ describe('hasAccessibleChild', () => {
       expect(hasAccessibleChild(element)).toBe(true);
     });
 
+    it('Returns true for JSXText Element', () => {
+      const child = {
+        type: 'JSXText',
+        value: 'foo',
+      };
+      const element = JSXElementMock('div', [], [child]);
+      expect(hasAccessibleChild(element)).toBe(true);
+    });
+
     it('Returns false for hidden child JSXElement', () => {
       const ariaHiddenAttr = JSXAttributeMock('aria-hidden', true);
       const child = JSXElementMock('div', [ariaHiddenAttr]);

src/util/hasAccessibleChild.js

@@ -8,6 +8,7 @@ export default function hasAccessibleChild(node: JSXElement): boolean {
   return node.children.some((child) => {
     switch (child.type) {
       case 'Literal':
+      case 'JSXText':
         return Boolean(child.value);
       case 'JSXElement':
         return !isHiddenFromScreenReader(