From 041f9ba137e6f2fa5302cff53ab5794365c83204 Mon Sep 17 00:00:00 2001
From: Wenbo Yu <wenboyu2@gmail.com>
Date: Tue, 16 Jan 2018 12:59:49 -0800
Subject: [PATCH] Fix: upload item action should show tooltip properly (#159)

---
 src/components/ContentUploader/ItemAction.js | 27 ++++++--------------
 1 file changed, 8 insertions(+), 19 deletions(-)

diff --git a/src/components/ContentUploader/ItemAction.js b/src/components/ContentUploader/ItemAction.js
index f2fc692fff..0fc4c02f4d 100644
--- a/src/components/ContentUploader/ItemAction.js
+++ b/src/components/ContentUploader/ItemAction.js
@@ -27,43 +27,32 @@ type Props = {
 
 const ItemAction = ({ status, onClick, intl, rootElement }: Props) => {
     let icon = <IconClose />;
-    let title = intl.formatMessage(messages.cancel);
-    let tooltip = '';
+    let tooltip = intl.formatMessage(messages.uploadsCancelButtonTooltip);
 
     switch (status) {
         case STATUS_COMPLETE:
             icon = <IconCheck color={ICON_CHECK_COLOR} />;
-            title = intl.formatMessage(messages.remove);
+            tooltip = intl.formatMessage(messages.remove);
             break;
         case STATUS_ERROR:
             icon = <IconRetry />;
-            title = intl.formatMessage(messages.retry);
+            tooltip = intl.formatMessage(messages.retry);
             break;
         case STATUS_IN_PROGRESS:
             icon = <IconInProgress />;
-            title = intl.formatMessage(messages.remove);
-            tooltip = intl.formatMessage(messages.uploadsCancelButtonTooltip);
             break;
         case STATUS_PENDING:
         default:
         // empty
     }
 
-    const button = (
-        <PlainButton type='button' onClick={onClick} title={title}>
-            {icon}
-        </PlainButton>
-    );
-
     return (
         <div className='bcu-item-action'>
-            {tooltip ? (
-                <Tooltip text={tooltip} position='top-left' bodyElement={rootElement}>
-                    {button}
-                </Tooltip>
-            ) : (
-                button
-            )}
+            <Tooltip text={tooltip} position='top-left' bodyElement={rootElement}>
+                <PlainButton type='button' onClick={onClick}>
+                    {icon}
+                </PlainButton>
+            </Tooltip>
         </div>
     );
 };