From 746a4d35ce071d4032d62fedb4ca0aaf649ed95f Mon Sep 17 00:00:00 2001 From: Jose Garcia Date: Thu, 17 Oct 2024 18:42:37 -0700 Subject: [PATCH] Make PR title a PR link, remove PR number column --- src/popup/components/FocusView.tsx | 13 ++++++++----- static/popup.css | 5 +---- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/popup/components/FocusView.tsx b/src/popup/components/FocusView.tsx index b54f279..228abff 100644 --- a/src/popup/components/FocusView.tsx +++ b/src/popup/components/FocusView.tsx @@ -44,22 +44,25 @@ const PullRequestRow = ({ userId, pullRequest, provider, draftCount = 0 }: PullR return ( <>
-
{pullRequest.title}
-
{pullRequest.repository.name}
-
+
{ + if (!pullRequest.url) { + return; + } + // Since there is a decent chance that the PR will be acted upon after the user clicks on it, // mark the focus view data as stale so that it will be refetched when the user returns. void queryClient.invalidateQueries({ queryKey: [userId, 'focusViewData', provider] }); }} title={`View pull request on ${ProviderMeta[provider].name}`} > - #{pullRequest.number} + #{pullRequest.number} {pullRequest.title}
+
{pullRequest.repository.name}
{deepLinkUrl && ( diff --git a/static/popup.css b/static/popup.css index b3a21ae..0f4d3fc 100644 --- a/static/popup.css +++ b/static/popup.css @@ -249,14 +249,11 @@ html { margin-right: 12px; } .focus-view .pull-request-bucket .pull-request .pull-request-title { - width: 274; + width: 350px; } .focus-view .pull-request-bucket .pull-request .repository-name { width: 148px; } -.focus-view .pull-request-bucket .pull-request .pull-request-number { - width: 64px; -} .focus-view .pull-request-bucket .pr-drafts-badge { border: 1px solid var(--color-border-input); border-radius: 12px;