fix(PoweredBy): add a label to the Algolia logo

algolia · Jul 28, 2017 · 2a6cf4f · 2a6cf4f
1 parent a46f6e4
commit 2a6cf4f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/packages/react-instantsearch/src/components/PoweredBy.js b/packages/react-instantsearch/src/components/PoweredBy.js
@@ -50,8 +50,17 @@ class PoweredBy extends Component {
 
     return (
       <div {...cx('root')}>
-        <span {...cx('searchBy')}>{translate('searchBy')} </span>
-        <a href={url} target="_blank" {...cx('algoliaLink')}><AlgoliaLogo /></a>
+        <span {...cx('searchBy')}>
+          {translate('searchBy')}{' '}
+        </span>
+        <a
+          href={url}
+          target="_blank"
+          {...cx('algoliaLink')}
+          ariaLabel="Algolia"
+        >
+          <AlgoliaLogo />
+        </a>
       </div>
     );
   }

diff --git a/packages/react-instantsearch/src/components/__snapshots__/PoweredBy.test.js.snap b/packages/react-instantsearch/src/components/__snapshots__/PoweredBy.test.js.snap
@@ -11,6 +11,7 @@ exports[`PoweredBy applies translations 1`] = `
      
   </span>
   <a
+    ariaLabel="Algolia"
     className="ais-PoweredBy__algoliaLink"
     href="url"
     target="_blank"
@@ -66,6 +67,7 @@ exports[`PoweredBy default 1`] = `
      
   </span>
   <a
+    ariaLabel="Algolia"
     className="ais-PoweredBy__algoliaLink"
     href="url"
     target="_blank"