From 39e05f16d7f9c4662bb2cc30ab76c57af00b1fe8 Mon Sep 17 00:00:00 2001
From: Matthew Boston <matthew@matthewboston.com>
Date: Fri, 15 Aug 2025 04:42:53 -0600
Subject: [PATCH] rename PR button handler and update button text for clarity
 (#484)

---
 frontend/src/components/tasks/Toolbar/CurrentAttempt.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/frontend/src/components/tasks/Toolbar/CurrentAttempt.tsx b/frontend/src/components/tasks/Toolbar/CurrentAttempt.tsx
index eae6c28c..4b90fdcc 100644
--- a/frontend/src/components/tasks/Toolbar/CurrentAttempt.tsx
+++ b/frontend/src/components/tasks/Toolbar/CurrentAttempt.tsx
@@ -321,10 +321,10 @@ function CurrentAttempt({
     setShowRebaseDialog(true);
   };
 
-  const handleCreatePRClick = async () => {
+  const handlePRButtonClick = async () => {
     if (!projectId || !selectedAttempt?.id || !selectedAttempt?.task_id) return;
 
-    // If PR already exists, open it
+    // If PR already exists, view it in a new tab
     if (selectedAttempt.pr_url) {
       window.open(selectedAttempt.pr_url, '_blank');
       return;
@@ -602,7 +602,7 @@ function CurrentAttempt({
                 !branchStatus.merged && (
                   <>
                     <Button
-                      onClick={handleCreatePRClick}
+                      onClick={handlePRButtonClick}
                       disabled={
                         creatingPR ||
                         Boolean(branchStatus.is_behind) ||
@@ -614,7 +614,7 @@ function CurrentAttempt({
                     >
                       <GitPullRequest className="h-3 w-3" />
                       {selectedAttempt.pr_url
-                        ? 'Open PR'
+                        ? 'View PR'
                         : creatingPR
                           ? 'Creating...'
                           : 'Create PR'}