From 538fc8e929e0af498fbd4680af56e6186ab026f8 Mon Sep 17 00:00:00 2001
From: apane <agustin.pane@gmail.com>
Date: Fri, 8 Nov 2019 16:00:29 -0300
Subject: [PATCH 1/2] Let non-owner users to edit the owners name

---
 .../Settings/ManageOwners/index.jsx           | 52 ++++++++++---------
 1 file changed, 27 insertions(+), 25 deletions(-)

diff --git a/src/routes/safe/components/Settings/ManageOwners/index.jsx b/src/routes/safe/components/Settings/ManageOwners/index.jsx
index 436788f0..7c8e877d 100644
--- a/src/routes/safe/components/Settings/ManageOwners/index.jsx
+++ b/src/routes/safe/components/Settings/ManageOwners/index.jsx
@@ -162,33 +162,35 @@ class ManageOwners extends React.Component<Props, State> {
                   </TableCell>
                 ))}
                 <TableCell component="td">
-                  {granted && (
-                    <Row align="end" className={classes.actions}>
-                      <Img
-                        alt="Edit owner"
-                        className={classes.editOwnerIcon}
-                        src={RenameOwnerIcon}
-                        onClick={this.onShow('EditOwner', row)}
-                        testId={RENAME_OWNER_BTN_TEST_ID}
-                      />
-                      <Img
-                        alt="Replace owner"
-                        className={classes.replaceOwnerIcon}
-                        src={ReplaceOwnerIcon}
-                        onClick={this.onShow('ReplaceOwner', row)}
-                        testId={REPLACE_OWNER_BTN_TEST_ID}
-                      />
-                      {ownerData.size > 1 && (
+                  <Row align="end" className={classes.actions}>
+                    <Img
+                      alt="Edit owner"
+                      className={classes.editOwnerIcon}
+                      src={RenameOwnerIcon}
+                      onClick={this.onShow('EditOwner', row)}
+                      testId={RENAME_OWNER_BTN_TEST_ID}
+                    />
+                    {granted && (
+                      <Row align="end" className={classes.actions}>
                         <Img
-                          alt="Remove owner"
-                          className={classes.removeOwnerIcon}
-                          src={RemoveOwnerIcon}
-                          onClick={this.onShow('RemoveOwner', row)}
-                          testId={REMOVE_OWNER_BTN_TEST_ID}
+                          alt="Replace owner"
+                          className={classes.replaceOwnerIcon}
+                          src={ReplaceOwnerIcon}
+                          onClick={this.onShow('ReplaceOwner', row)}
+                          testId={REPLACE_OWNER_BTN_TEST_ID}
                         />
-                      )}
-                    </Row>
-                  )}
+                        {ownerData.size > 1 && (
+                          <Img
+                            alt="Remove owner"
+                            className={classes.removeOwnerIcon}
+                            src={RemoveOwnerIcon}
+                            onClick={this.onShow('RemoveOwner', row)}
+                            testId={REMOVE_OWNER_BTN_TEST_ID}
+                          />
+                        )}
+                      </Row>
+                    ) }
+                  </Row>
                 </TableCell>
               </TableRow>
             ))}

From 141864333fe7cafa4e6910ccdebc337d34b486b7 Mon Sep 17 00:00:00 2001
From: apane <agustin.pane@gmail.com>
Date: Wed, 13 Nov 2019 10:21:19 -0300
Subject: [PATCH 2/2] Fixs tags

---
 src/routes/safe/components/Settings/ManageOwners/index.jsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/routes/safe/components/Settings/ManageOwners/index.jsx b/src/routes/safe/components/Settings/ManageOwners/index.jsx
index 7c8e877d..a9927332 100644
--- a/src/routes/safe/components/Settings/ManageOwners/index.jsx
+++ b/src/routes/safe/components/Settings/ManageOwners/index.jsx
@@ -171,7 +171,7 @@ class ManageOwners extends React.Component<Props, State> {
                       testId={RENAME_OWNER_BTN_TEST_ID}
                     />
                     {granted && (
-                      <Row align="end" className={classes.actions}>
+                      <>
                         <Img
                           alt="Replace owner"
                           className={classes.replaceOwnerIcon}
@@ -188,7 +188,7 @@ class ManageOwners extends React.Component<Props, State> {
                             testId={REMOVE_OWNER_BTN_TEST_ID}
                           />
                         )}
-                      </Row>
+                      </>
                     ) }
                   </Row>
                 </TableCell>