Fix quick links key warning (#3049)

This was giving a warning that the <td> should have a key.

I looked at the code and it seemed stupid to have the <td>
in the QuickLinks return to begin with, so I changed it to return
just the links. That solves the warning, but it probably shouldn't
(possibly a React bug?) so to be actually safe, this also creates
a React fragment to return instead of the array.

root/edit/components/ListHeader.js

@@ -22,13 +22,13 @@ type Props = {
   +username?: string,
 };
 
-const QuickLinksSection = ({
+const QuickLinks = ({
   entity,
   isSearch = false,
   page,
   refineUrlArgs,
   username,
-}: Props): React$Element<'td'> => {
+}: Props): React$Element<React$FragmentType> => {
   const $c = React.useContext(CatalystContext);
   const isSecureConnection = $c.req.secure;
   const protocol = isSecureConnection ? 'https://' : 'http://';
@@ -173,16 +173,14 @@ const QuickLinksSection = ({
       </a>,
     );
   }
-  return (
-    <td>
-      {quickLinks.reduce((accum: Array<React$Node>, link, index) => {
-        accum.push(link);
-        if (index < (quickLinks.length - 1)) {
-          accum.push(' | ');
-        }
-        return accum;
-      }, [])}
-    </td>
+  return React.createElement(React.Fragment, null, ...quickLinks.reduce(
+    (accum: Array<React$Node>, link, index) => {
+      accum.push(link);
+      if (index < (quickLinks.length - 1)) {
+        accum.push(' | ');
+      }
+      return accum;
+    }, [])
   );
 };
 
@@ -198,13 +196,15 @@ const ListHeader = ({
       <th>
         {l('Quick links:')}
       </th>
-      <QuickLinksSection
-        entity={entity}
-        isSearch={isSearch}
-        page={page}
-        refineUrlArgs={refineUrlArgs}
-        username={username}
-      />
+      <td>
+        <QuickLinks
+          entity={entity}
+          isSearch={isSearch}
+          page={page}
+          refineUrlArgs={refineUrlArgs}
+          username={username}
+        />
+      </td>
     </tr>
     <tr>
       <th>