From 421f5903d7f75897dfe2f35e70a2220634e89fbb Mon Sep 17 00:00:00 2001
From: Tan Nhu <tan@harness.io>
Date: Wed, 6 Mar 2024 08:43:15 +0000
Subject: [PATCH] Adjust full diff icon size and stroke for clarification
 (#1104)

---
 web/src/components/DiffViewer/DiffViewer.tsx | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/web/src/components/DiffViewer/DiffViewer.tsx b/web/src/components/DiffViewer/DiffViewer.tsx
index ac6e6f5f9..6dc4f6034 100644
--- a/web/src/components/DiffViewer/DiffViewer.tsx
+++ b/web/src/components/DiffViewer/DiffViewer.tsx
@@ -335,6 +335,8 @@ const DiffViewerInternal: React.FC<DiffViewerProps> = ({
     }
   }, [useFullDiff, memorizedState, diff.filePath, cancelGetFullDiff, getFullDiff, fullDiffLoading])
 
+  const ToggleFullDiffIcon = useMemo(() => (useFullDiff ? Collapse : Expand), [useFullDiff])
+
   return (
     <Container
       ref={containerRef}
@@ -351,7 +353,7 @@ const DiffViewerInternal: React.FC<DiffViewerProps> = ({
               size={ButtonSize.SMALL}
               onClick={() => setCollapsed(!collapsed)}
               iconProps={{
-                size: 10,
+                size: 12,
                 style: {
                   color: '#383946',
                   flexGrow: 1,
@@ -366,7 +368,7 @@ const DiffViewerInternal: React.FC<DiffViewerProps> = ({
               className={css.expandCollapseDiffBtn}
               onClick={toggleFullDiff}
               title={getString(useFullDiff ? 'pr.collapseFullFile' : 'pr.expandFullFile')}>
-              {useFullDiff ? <Collapse width="16" height="16" /> : <Expand width="16" height="16" />}
+              <ToggleFullDiffIcon width="16" height="16" strokeWidth="2" />
             </Button>
             <Text
               inline