From a2e937a36d3adbf8897c04bc42a1f74936549aad Mon Sep 17 00:00:00 2001
From: Andrew Golovanov <spacewebdeveloper@gmail.com>
Date: Wed, 15 Jan 2025 17:38:04 +0100
Subject: [PATCH] fix: minor changes

---
 .../components/pull-request-header.tsx        |  2 +-
 .../conversation/pull-request-panel.tsx       | 21 +++++++------------
 .../ui/src/views/repo/pull-request/utils.ts   |  4 ++--
 3 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/packages/ui/src/views/repo/pull-request/components/pull-request-header.tsx b/packages/ui/src/views/repo/pull-request/components/pull-request-header.tsx
index 7a96deaee..93614c00f 100644
--- a/packages/ui/src/views/repo/pull-request/components/pull-request-header.tsx
+++ b/packages/ui/src/views/repo/pull-request/components/pull-request-header.tsx
@@ -7,7 +7,7 @@ import { timeAgo } from '@utils/utils'
 import { IconType } from '../pull-request.types'
 import { getPrState } from '../utils'
 
-type ThemeType = 'default' | 'destructive' | 'warning' | 'success' | 'emphasis' | 'muted' | null | undefined
+type ThemeType = 'default' | 'destructive' | 'success' | 'emphasis' | 'muted' | null | undefined
 interface PullRequestTitleProps {
   data: {
     title?: string
diff --git a/packages/ui/src/views/repo/pull-request/details/components/conversation/pull-request-panel.tsx b/packages/ui/src/views/repo/pull-request/details/components/conversation/pull-request-panel.tsx
index e54e07fe5..2e16c50c8 100644
--- a/packages/ui/src/views/repo/pull-request/details/components/conversation/pull-request-panel.tsx
+++ b/packages/ui/src/views/repo/pull-request/details/components/conversation/pull-request-panel.tsx
@@ -11,7 +11,6 @@ import {
   DropdownMenuItem,
   DropdownMenuRadioGroup,
   DropdownMenuRadioItem,
-  DropdownMenuShortcut,
   DropdownMenuTrigger,
   Icon,
   Layout,
@@ -202,10 +201,12 @@ const PullRequestPanel = ({
       setNotBypassable(checkIfBypassAllowed)
     }
   }, [ruleViolationArr])
+
   const rebasePossible = useMemo(
     () => pullReqMetadata?.merge_target_sha !== pullReqMetadata?.merge_base_sha && !pullReqMetadata?.merged,
     [pullReqMetadata]
   )
+
   const moreTooltip = () => {
     return (
       <DropdownMenu>
@@ -214,7 +215,7 @@ const PullRequestPanel = ({
             <Icon name="vertical-ellipsis" size={12} />
           </Button>
         </DropdownMenuTrigger>
-        <DropdownMenuContent className="w-[180px] rounded-[10px] border border-borders-1 bg-background-2 py-2 shadow-sm">
+        <DropdownMenuContent className="w-[200px]" align="end">
           <DropdownMenuGroup>
             <DropdownMenuItem
               onClick={e => {
@@ -222,20 +223,16 @@ const PullRequestPanel = ({
 
                 e.stopPropagation()
               }}
-              className="cursor-pointer"
             >
-              <DropdownMenuShortcut className="ml-0"></DropdownMenuShortcut>
-              {'Mark as draft'}
+              Mark as draft
             </DropdownMenuItem>
             <DropdownMenuItem
               onClick={e => {
                 handlePrState('closed')
                 e.stopPropagation()
               }}
-              className="cursor-pointer"
             >
-              <DropdownMenuShortcut className="ml-0"></DropdownMenuShortcut>
-              {'Close pull request'}
+              Close pull request
             </DropdownMenuItem>
             {rebasePossible && (
               <DropdownMenuItem
@@ -243,10 +240,8 @@ const PullRequestPanel = ({
                   handleRebaseBranch()
                   e.stopPropagation()
                 }}
-                className="cursor-pointer"
               >
-                <DropdownMenuShortcut className="ml-0"></DropdownMenuShortcut>
-                {'Rebase'}
+                Rebase
               </DropdownMenuItem>
             )}
           </DropdownMenuGroup>
@@ -305,7 +300,7 @@ const PullRequestPanel = ({
                   )}
                   <Button
                     variant={actions && !pullReqMetadata?.closed ? 'split' : 'default'}
-                    size={actions && !pullReqMetadata?.closed ? 'xs_split' : 'xs'}
+                    size={actions && !pullReqMetadata?.closed ? 'md_split' : 'md'}
                     theme={
                       mergeable && !ruleViolation && !pullReqMetadata?.is_draft && !pullReqMetadata?.closed
                         ? 'success'
@@ -313,7 +308,7 @@ const PullRequestPanel = ({
                           ? 'primary'
                           : checksInfo.status === 'pending' || checksInfo.status === 'running'
                             ? 'warning'
-                            : 'error'
+                            : 'disabled'
                     }
                     disabled={!checkboxBypass && ruleViolation}
                     onClick={actions[0]?.action}
diff --git a/packages/ui/src/views/repo/pull-request/utils.ts b/packages/ui/src/views/repo/pull-request/utils.ts
index f2a282530..e1e050137 100644
--- a/packages/ui/src/views/repo/pull-request/utils.ts
+++ b/packages/ui/src/views/repo/pull-request/utils.ts
@@ -6,11 +6,11 @@ export const getPrState = (
   state?: string
 ): { icon: IconType; text: string; theme: string } => {
   if (state === 'open' && is_draft) {
-    return { icon: 'pr-draft', text: 'Draft', theme: 'warning' }
+    return { icon: 'pr-draft', text: 'Draft', theme: 'muted' }
   } else if (merged) {
     return { icon: 'pr-merge', text: 'Merged', theme: 'emphasis' }
   } else if (state === 'closed') {
-    return { icon: 'pr-closed', text: 'Closed', theme: 'muted' }
+    return { icon: 'pr-closed', text: 'Closed', theme: 'destructive' }
   } else {
     return { icon: 'pr-open', text: 'Open', theme: 'success' }
   }