From 6a29fd787e6a041ebf2634d60a659b99f23880c9 Mon Sep 17 00:00:00 2001
From: Freya Gustavsson <freya@spytec.se>
Date: Mon, 10 Jun 2024 13:30:33 +0200
Subject: [PATCH] fix: add external link to Pipelines table

---
 frontend/src/app/Pipelines/PipelinesTable.tsx | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/frontend/src/app/Pipelines/PipelinesTable.tsx b/frontend/src/app/Pipelines/PipelinesTable.tsx
index ab92d7f..b60163e 100644
--- a/frontend/src/app/Pipelines/PipelinesTable.tsx
+++ b/frontend/src/app/Pipelines/PipelinesTable.tsx
@@ -22,6 +22,7 @@ import coprLogo from "../../static/copr.ico";
 import kojiLogo from "../../static/koji.ico";
 import { useInfiniteQuery, useQueryClient } from "@tanstack/react-query";
 import { Link } from "react-router-dom";
+import { ExternalLinkAltIcon } from "@patternfly/react-icons";
 
 interface StatusItem {
   packit_id: number;
@@ -123,10 +124,10 @@ const PipelinesTable = () => {
 
   // Headings
   const columns = [
-    { title: "" }, // space for forge icon
-    { title: "Trigger", transforms: [cellWidth(15)] },
+    { title: "Trigger", transforms: [cellWidth(20)] },
     { title: "Time Submitted", transforms: [cellWidth(10)] },
-    { title: "Jobs", transforms: [cellWidth(70)] },
+    { title: "Jobs", transforms: [cellWidth(60)] },
+    { title: "External" },
   ];
 
   // Fetch data from dashboard backend (or if we want, directly from the API)
@@ -156,12 +157,10 @@ const PipelinesTable = () => {
     res.forEach((run) => {
       const singleRow = {
         cells: [
-          {
-            title: <ForgeIcon url={run.trigger.git_repo} />,
-          },
           {
             title: (
               <strong>
+                <ForgeIcon url={run.trigger.git_repo} />{" "}
                 <Link to={`/pipelines/${run.merged_run_id}`}>
                   <TriggerSuffix trigger={run.trigger} />
                 </Link>
@@ -217,6 +216,13 @@ const PipelinesTable = () => {
               </>
             ),
           },
+          {
+            title: (
+              <TriggerLink trigger={run.trigger}>
+                <ExternalLinkAltIcon />
+              </TriggerLink>
+            ),
+          },
         ],
       };
       rowsList.push(singleRow);