Remove double navigation in the experiments list (#2270)

growthbook · Apr 3, 2024 · 13ce00c · 13ce00c
1 parent 3c4ccc6
commit 13ce00c
Showing 1 changed file with 41 additions and 47 deletions.
diff --git a/packages/front-end/pages/experiments/index.tsx b/packages/front-end/pages/experiments/index.tsx
@@ -355,55 +355,49 @@ const ExperimentsPage = (): React.ReactElement => {
                         type="icon"
                       />
                     </td>
-                    <td
-                      onClick={() => {
-                        router.push(`/experiment/${e.id}`);
-                      }}
-                      className="cursor-pointer"
-                      data-title="Experiment name:"
-                    >
-                      <div className="d-flex flex-column">
-                        <div className="d-flex">
-                          <Link
-                            href={`/experiment/${e.id}`}
-                            className="testname"
-                          >
-                            {e.name}
-                          </Link>
-                          {e.hasVisualChangesets ? (
-                            <Tooltip
-                              className="d-flex align-items-center ml-2"
-                              body="Visual experiment"
-                            >
-                              <RxDesktop className="text-blue" />
-                            </Tooltip>
-                          ) : null}
-                          {(e.linkedFeatures || []).length > 0 ? (
-                            <Tooltip
-                              className="d-flex align-items-center ml-2"
-                              body="Linked Feature Flag"
-                            >
-                              <BsFlag className="text-blue" />
-                            </Tooltip>
-                          ) : null}
-                          {e.hasURLRedirects ? (
-                            <Tooltip
-                              className="d-flex align-items-center ml-2"
-                              body="URL Redirect experiment"
+                    <td data-title="Experiment name:" className="p-0">
+                      <Link
+                        href={`/experiment/${e.id}`}
+                        className="d-block p-2"
+                      >
+                        <div className="d-flex flex-column">
+                          <div className="d-flex">
+                            <span className="testname">{e.name}</span>
+                            {e.hasVisualChangesets ? (
+                              <Tooltip
+                                className="d-flex align-items-center ml-2"
+                                body="Visual experiment"
+                              >
+                                <RxDesktop className="text-blue" />
+                              </Tooltip>
+                            ) : null}
+                            {(e.linkedFeatures || []).length > 0 ? (
+                              <Tooltip
+                                className="d-flex align-items-center ml-2"
+                                body="Linked Feature Flag"
+                              >
+                                <BsFlag className="text-blue" />
+                              </Tooltip>
+                            ) : null}
+                            {e.hasURLRedirects ? (
+                              <Tooltip
+                                className="d-flex align-items-center ml-2"
+                                body="URL Redirect experiment"
+                              >
+                                <PiShuffle className="text-blue" />
+                              </Tooltip>
+                            ) : null}
+                          </div>
+                          {isFiltered && e.trackingKey && (
+                            <span
+                              className="testid text-muted small"
+                              title="Experiment Id"
                             >
-                              <PiShuffle className="text-blue" />
-                            </Tooltip>
-                          ) : null}
+                              {e.trackingKey}
+                            </span>
+                          )}
                         </div>
-                        {isFiltered && e.trackingKey && (
-                          <span
-                            className="testid text-muted small"
-                            title="Experiment Id"
-                          >
-                            {e.trackingKey}
-                          </span>
-                        )}
-                      </div>
+                      </Link>
                     </td>
                     {showProjectColumn && (
                       <td className="nowrap" data-title="Project:">