Fix for issues #33 #19 (#41)

* Added check for title if 2F returns empty string

* Implemented requested changes

bundle.js

@@ -660,7 +660,8 @@ var accname = (function (exports) {
             .join(' ');
         // #SPEC_ASSUMPTION (F.2) : that CSS generated content should be
         // concatenated to accumulatedText
-        return (cssBeforeContent + accumulatedText + cssAfterContent).trim();
+        const result = (cssBeforeContent + accumulatedText + cssAfterContent).trim();
+        return result || null;
     }
 
     /**

src/lib/compute_text_alternative_test.ts

@@ -83,4 +83,26 @@ describe('The computeTextAlternative function', () => {
       rulesApplied: new Set<Rule>(['2B', '2F', '2G']),
     });
   });
+
+  it('check title attribute for name when subtree is empty', () => {
+    render(
+      html` <input type="checkbox" title="Hello world" id="foo" /> `,
+      container
+    );
+    const elem = document.getElementById('foo')!;
+    expect(computeTextAlternative(elem).name).toEqual('Hello world');
+  });
+
+  it('check title attribute for name when subtree is hidden', () => {
+    render(
+      html`
+        <button title="Hello world" id="foo">
+          <div aria-hidden="true">Invisible text</div>
+        </button>
+      `,
+      container
+    );
+    const elem = document.getElementById('foo')!;
+    expect(computeTextAlternative(elem).name).toEqual('Hello world');
+  });
 });

src/lib/rule2F.ts

@@ -228,5 +228,7 @@ export function rule2F(
 
   // #SPEC_ASSUMPTION (F.2) : that CSS generated content should be
   // concatenated to accumulatedText
-  return (cssBeforeContent + accumulatedText + cssAfterContent).trim();
+  const result = (cssBeforeContent + accumulatedText + cssAfterContent).trim();
+
+  return result || null;
 }