Rewards: Imported from brave-ui: Addressing code review

brave · Sep 19, 2019 · b92362d · b92362d
1 parent 9e5904a
commit b92362d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/components/brave_rewards/resources/ui/components/walletWrapper/index.tsx b/components/brave_rewards/resources/ui/components/walletWrapper/index.tsx
@@ -246,8 +246,8 @@ export default class WalletWrapper extends React.PureComponent<Props, State> {
   }
 
   getNotificationButton = (type: NotificationType, onClose: any) => {
-    let buttonText = 'OK'
-    let buttonAction = onClose
+    let buttonText
+    let buttonAction
 
     switch (type) {
       case 'grant':
@@ -258,6 +258,10 @@ export default class WalletWrapper extends React.PureComponent<Props, State> {
         buttonText = getLocale('backupNow')
         buttonAction = this.onNotificationClick
         break
+      default:
+        buttonText = getLocale('ok').toUpperCase()
+        buttonAction = onClose
+        break
     }
 
     return (
@@ -286,7 +290,7 @@ export default class WalletWrapper extends React.PureComponent<Props, State> {
         <StyledNotificationContent>
           {this.getNotificationIcon(notification)}
           {this.getNotificationMessage(notification)}
-          <StyledButtonWrapper type={notification.type}>
+          <StyledButtonWrapper>
             {this.getNotificationButton(notification.type, onClose)}
           </StyledButtonWrapper>
         </StyledNotificationContent>

diff --git a/components/brave_rewards/resources/ui/components/walletWrapper/style.ts b/components/brave_rewards/resources/ui/components/walletWrapper/style.ts
@@ -13,8 +13,6 @@ interface StyledProps {
   compact?: boolean,
   background?: string,
   isMobile?: boolean,
-  type?: string,
-  onClick?: Function | undefined,
   notification?: Notification | undefined
 }
 
@@ -283,7 +281,6 @@ export const StyledTypeText = styled<StyledProps, 'span'>('span')`
   font-weight: 500;
   margin-right: 5px;
   display: inline-block;
-  cursor: ${p => p.onClick ? 'pointer' : 'default'};
 `
 
 export const StyledMessageText = styled<StyledProps, 'span'>('span')`