Fix accessibility (#640)

Signed-off-by: Matt Provost <[email protected]> Co-authored-by: Iskander Rakhmanberdiyev <[email protected]> (cherry picked from commit 18cc9df) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Jun 27, 2023 · 7ce7189 · 7ce7189
1 parent e6a2842
commit 7ce7189
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src-docs/src/assets/version-selector.js b/src-docs/src/assets/version-selector.js
@@ -127,7 +127,7 @@ const tpl = `
     </style>
     <a id="root" role="button" aria-labelledby="selected" aria-controls="dropdown" tabindex="0">
       <svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" role="img" 
-       width="1em" height="1em" preserveAspectRatio="xMidYMid meet" viewBox="0 0 24 24">
+       width="1em" height="1em" preserveAspectRatio="xMidYMid meet" viewBox="0 0 24 24" aria-hidden="true">
        <g fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round">
       <path d="M6 9l6 6l6-6"/></g>
       </svg>

diff --git a/src-docs/src/views/suggest/global_filter_item.js b/src-docs/src/views/suggest/global_filter_item.js
@@ -99,7 +99,7 @@ export const GlobalFilterItem = (props) => {
       closeButtonProps={{
         // Removing tab focus on close button because the same option can be optained through the context menu
         // Also, we may want to add a `DEL` keyboard press functionality
-        tabIndex: '-1',
+        tabIndex: -1,
       }}
       {...rest}>
       {prefix}

diff --git a/src-docs/src/views/tool_tip/tool_tip.js b/src-docs/src/views/tool_tip/tool_tip.js
@@ -60,7 +60,7 @@ export default () => (
       <p>
         This tooltip appears on the bottom of this icon:{' '}
         <OuiToolTip position="bottom" content="Here is some tooltip text">
-          <OuiIcon tabIndex="0" type="alert" />
+          <OuiIcon tabIndex="0" type="alert" aria-label="Alert" />
         </OuiToolTip>
       </p>
     </OuiText>