Changing color to hex code, making action order consistent

codeforpdx · Nov 14, 2024 · ef5ac78 · ef5ac78
1 parent 82a0ecd
commit ef5ac78
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 22 deletions.
diff --git a/src/components/Contacts/ContactListTableDesktop.jsx b/src/components/Contacts/ContactListTableDesktop.jsx
@@ -74,11 +74,6 @@ const ContactListTableDesktop = ({
     },
     {
       field: 'webId',
-      headerName: 'Web ID',
-      minWidth: 150,
-      flex: 1,
-      headerAlign: 'center',
-      align: 'center',
       renderCell: (contact) => (
         <Typography
           noWrap
@@ -87,7 +82,12 @@ const ContactListTableDesktop = ({
         >
           {contact.id}
         </Typography>
-      )
+      ),
+      headerName: 'Web ID',
+      minWidth: 150,
+      flex: 1,
+      headerAlign: 'center',
+      align: 'center'
     },
     {
       field: 'Profile',

diff --git a/src/components/Contacts/ContactListTableMobile.jsx b/src/components/Contacts/ContactListTableMobile.jsx
@@ -185,6 +185,18 @@ const ContactListTableMobile = ({
                   'aria-labelledby': 'actions-icon-button'
                 }}
               >
+                <MenuItem
+                  component={Button}
+                  onClick={handleMenuItemClick(
+                    () =>
+                      saveToClipboard(contact.webId, 'webId copied to clipboard', addNotification),
+                    contact
+                  )}
+                  startIcon={<ContentCopyIcon sx={iconSize} />}
+                  sx={iconStyling}
+                >
+                  Copy WebId
+                </MenuItem>
                 <MenuItem
                   component={Button}
                   onClick={handleMenuItemClick(handleProfileClick, contact)}
@@ -209,21 +221,6 @@ const ContactListTableMobile = ({
                 >
                   Message
                 </MenuItem>
-                {/* TODO: Keep copy function? */}
-                {/* If so, also add to Desktop table? */}
-                {/* Maybe without any icon. Simply click on the web ID and it will copy? */}
-                <MenuItem
-                  component={Button}
-                  onClick={handleMenuItemClick(
-                    () =>
-                      saveToClipboard(contact.webId, 'webId copied to clipboard', addNotification),
-                    contact
-                  )}
-                  startIcon={<ContentCopyIcon sx={iconSize} />}
-                  sx={iconStyling}
-                >
-                  Copy WebId
-                </MenuItem>
                 <MenuItem
                   component={Button}
                   onClick={handleMenuItemClick(editContact, contact)}

diff --git a/src/pages/Contacts.jsx b/src/pages/Contacts.jsx
@@ -158,7 +158,6 @@ const Contacts = () => {
                 }}
                 IconComponent={KeyboardArrowDownIcon}
               >
-                {' '}
                 <MenuItem value="Sort by:" disabled>
                   Sort by:
                 </MenuItem>