Update aria-labels for header buttons with notification

elastic · Sep 14, 2020 · c2fc811 · c2fc811
1 parent 901a2d0
commit c2fc811
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src-docs/src/views/header/header_dark.tsx b/src-docs/src/views/header/header_dark.tsx
@@ -51,7 +51,7 @@ export default ({ theme }: { theme: any }) => (
             Production logs
           </EuiBadge>,
           <EuiHeaderSectionItemButton
-            aria-label="Notifications"
+            aria-label="2 Notifications"
             notification={'2'}>
             <EuiIcon type="cheer" size="m" />
           </EuiHeaderSectionItemButton>,

diff --git a/src-docs/src/views/header/header_elastic_pattern.js b/src-docs/src/views/header/header_elastic_pattern.js
@@ -299,8 +299,8 @@ export default ({ theme }) => {
                 items: [
                   <EuiShowFor sizes={['xs', 's']}>{search}</EuiShowFor>,
                   <EuiHeaderSectionItemButton
-                    aria-label="Notifications"
                     notification={true}
+                    aria-label="Notifictations: Updates available"
                     onClick={() =>
                       setIsAlertFlyoutVisible(!isAlertFlyoutVisible)
                     }>

diff --git a/src-docs/src/views/header/header_updates.js b/src-docs/src/views/header/header_updates.js
@@ -117,7 +117,7 @@ export default ({ flyoutOrPopover = 'flyout' }) => {
       aria-expanded={isFlyoutVisible}
       aria-haspopup="true"
       aria-label={`News feed: ${
-        { showBadge } ? 'Updates available' : 'No updates'
+        showBadge ? 'Updates available' : 'No updates'
       }`}
       onClick={() => showFlyout()}
       notification={showBadge}>