Fix #596: Notification when safe is already updated (#599)

* Fix notification of update if the safe is already updated * Makes the notification clickable Displays the notification for owners only * Identify upgrade tx * Add red badge to Settings tab * Fixs Padding Removes the red dot if the user is not an owner Co-authored-by: Fernando <[email protected]>
5afe · Mar 9, 2020 · 37a8a01 · 37a8a01
1 parent d3bac2f
commit 37a8a01
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/src/routes/safe/components/Layout.jsx b/src/routes/safe/components/Layout.jsx
@@ -166,7 +166,7 @@ const Layout = (props: Props) => {
       Apps
     </>
   )
-
+  
   const labelSettings = (
     <>
       <SettingsIcon />

diff --git a/src/routes/safe/components/Settings/index.jsx b/src/routes/safe/components/Settings/index.jsx
@@ -143,13 +143,7 @@ class Settings extends React.Component<Props, State> {
                 onClick={this.handleChange(1)}
               >
                 <SafeDetailsIcon />
-                <Badge
-                  badgeContent=" "
-                  variant="dot"
-                  invisible={!this.state.needUpdate || !granted}
-                  color="error"
-                  style={{ paddingRight: '10px' }}
-                >
+                <Badge badgeContent=" " variant="dot" invisible={!this.state.needUpdate || !granted} color="error" style={{paddingRight: '10px'}}>
                   Safe details
                 </Badge>
               </Row>