From 85b9e5168dc4c868ef116e7044ab026e5c982e06 Mon Sep 17 00:00:00 2001
From: Christos Nasikas <christos.nasikas@elastic.co>
Date: Tue, 31 Mar 2020 13:29:07 +0300
Subject: [PATCH] Fix conflicts and bugs

---
 .../pages/case/components/configure_cases/button.tsx  |  1 +
 .../public/pages/case/components/tag_list/index.tsx   |  2 +-
 .../public/pages/case/components/user_list/index.tsx  | 11 +++++++----
 3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/x-pack/legacy/plugins/siem/public/pages/case/components/configure_cases/button.tsx b/x-pack/legacy/plugins/siem/public/pages/case/components/configure_cases/button.tsx
index 9cfc51da22e87..b0bea83148bda 100644
--- a/x-pack/legacy/plugins/siem/public/pages/case/components/configure_cases/button.tsx
+++ b/x-pack/legacy/plugins/siem/public/pages/case/components/configure_cases/button.tsx
@@ -31,6 +31,7 @@ const ConfigureCaseButtonComponent: React.FC<ConfigureCaseButtonProps> = ({
         href={getConfigureCasesUrl(urlSearch)}
         iconType="controlsHorizontal"
         isDisabled={isDisabled}
+        aria-label={label}
       >
         {label}
       </EuiButton>
diff --git a/x-pack/legacy/plugins/siem/public/pages/case/components/tag_list/index.tsx b/x-pack/legacy/plugins/siem/public/pages/case/components/tag_list/index.tsx
index 0bfb0a4cb8831..f7d890ca60b16 100644
--- a/x-pack/legacy/plugins/siem/public/pages/case/components/tag_list/index.tsx
+++ b/x-pack/legacy/plugins/siem/public/pages/case/components/tag_list/index.tsx
@@ -66,7 +66,7 @@ export const TagList = React.memo(
             <EuiFlexItem grow={false}>
               <EuiButtonIcon
                 isDisabled={disabled}
-                aria-label={'tags'}
+                aria-label={i18n.EDIT_TAGS_ARIA}
                 iconType={'pencil'}
                 onClick={setIsEditTags.bind(null, true)}
               />
diff --git a/x-pack/legacy/plugins/siem/public/pages/case/components/user_list/index.tsx b/x-pack/legacy/plugins/siem/public/pages/case/components/user_list/index.tsx
index e9865baa1cd0e..579e8e48fa147 100644
--- a/x-pack/legacy/plugins/siem/public/pages/case/components/user_list/index.tsx
+++ b/x-pack/legacy/plugins/siem/public/pages/case/components/user_list/index.tsx
@@ -5,6 +5,8 @@
  */
 
 import React, { useCallback } from 'react';
+import { isEmpty } from 'lodash/fp';
+
 import {
   EuiButtonIcon,
   EuiText,
@@ -15,9 +17,10 @@ import {
   EuiLoadingSpinner,
   EuiToolTip,
 } from '@elastic/eui';
+
 import styled, { css } from 'styled-components';
-import { ElasticUser } from '../../../../containers/case/types';
 
+import { ElasticUser } from '../../../../containers/case/types';
 import * as i18n from './translations';
 
 interface UserListProps {
@@ -52,7 +55,7 @@ const renderUsers = (
             <MyAvatar name={fullName ? fullName : username ?? ''} />
           </EuiFlexItem>
           <EuiFlexItem>
-            <EuiToolTip position="top" content={<p>{fullName ?? username}</p>}>
+            <EuiToolTip position="top" content={<p>{fullName ? fullName : username ?? ''}</p>}>
               <p>
                 <strong>
                   <small data-test-subj="case-view-username">{username}</small>
@@ -67,8 +70,8 @@ const renderUsers = (
           data-test-subj="user-list-email-button"
           onClick={handleSendEmail.bind(null, email)}
           iconType="email"
-          aria-label="email"
-          isDisabled={email == null}
+          aria-label={i18n.SEND_EMAIL_ARIA(fullName ? fullName : username ?? '')}
+          isDisabled={isEmpty(email)}
         />
       </EuiFlexItem>
     </MyFlexGroup>