From 266c5b58b0fe28e6a1e90fc802ecaa669f34afbf Mon Sep 17 00:00:00 2001
From: Adam Setch <adam.setch@outlook.com>
Date: Mon, 11 Mar 2024 14:57:06 -0400
Subject: [PATCH 1/2] refactor(icons): replace refresh icon with octicons prime
 version.  switch default check-suite case to use rocket icon

---
 src/components/Sidebar.tsx   |  5 ++---
 src/icons/Refresh.tsx        | 26 --------------------------
 src/utils/github-api.test.ts |  2 +-
 src/utils/github-api.ts      |  3 +--
 4 files changed, 4 insertions(+), 32 deletions(-)
 delete mode 100644 src/icons/Refresh.tsx

diff --git a/src/components/Sidebar.tsx b/src/components/Sidebar.tsx
index 38574a812..f7610e23b 100644
--- a/src/components/Sidebar.tsx
+++ b/src/components/Sidebar.tsx
@@ -1,4 +1,4 @@
-import { BellIcon } from '@primer/octicons-react';
+import { BellIcon, SyncIcon } from '@primer/octicons-react';
 import { ipcRenderer } from 'electron';
 import React, { useCallback, useContext, useMemo } from 'react';
 import { useNavigate, useLocation } from 'react-router-dom';
@@ -7,7 +7,6 @@ import { Logo } from '../components/Logo';
 import { AppContext } from '../context/App';
 import { IconCog } from '../icons/Cog';
 import { IconQuit } from '../icons/Quit';
-import { IconRefresh } from '../icons/Refresh';
 import { Constants } from '../utils/constants';
 import { openExternalLink } from '../utils/comms';
 
@@ -72,7 +71,7 @@ export const Sidebar: React.FC = () => {
               }}
               aria-label="Refresh Notifications"
             >
-              <IconRefresh className="w-3.5 h-3.5" />
+              <SyncIcon size={18} />
             </button>
 
             <button
diff --git a/src/icons/Refresh.tsx b/src/icons/Refresh.tsx
deleted file mode 100644
index 605b459c9..000000000
--- a/src/icons/Refresh.tsx
+++ /dev/null
@@ -1,26 +0,0 @@
-import * as React from 'react';
-
-export const IconRefresh = ({ className = '' }: { className?: string }) => {
-  return (
-    <svg
-      aria-hidden="true"
-      className={className}
-      xmlns="http://www.w3.org/2000/svg"
-      viewBox="0 0 512 512"
-    >
-      <g className="prefix__fa-group">
-        <path
-          className="prefix__fa-secondary"
-          fill="currentColor"
-          d="M0 500V299.67a12 12 0 0112-12h200.33a12 12 0 0112 12v47.41a12 12 0 01-12.57 12l-101.87-4.88a176.07 176.07 0 00317.25-56.94 12 12 0 0111.67-9.26h49.09a12 12 0 0111.8 14.18C478.07 417.08 377.19 504 256 504a247.43 247.43 0 01-188.76-87.17l4.13 82.57a12 12 0 01-12 12.6H12a12 12 0 01-12-12z"
-          opacity={0.4}
-        />
-        <path
-          className="prefix__fa-primary"
-          fill="currentColor"
-          d="M12.3 209.82C33.93 94.92 134.81 8 256 8a247.4 247.4 0 01188.9 87.34l-4-82.77A12 12 0 01452.92 0h47.41a12 12 0 0112 12v200.33a12 12 0 01-12 12H300a12 12 0 01-12-12v-47.41a12 12 0 0112.57-12l101.53 4.88a176.07 176.07 0 00-317.24 56.94A12 12 0 0173.19 224H24.1a12 12 0 01-11.8-14.18z"
-        />
-      </g>
-    </svg>
-  );
-};
diff --git a/src/utils/github-api.test.ts b/src/utils/github-api.test.ts
index dbb5fe929..daeca7ca3 100644
--- a/src/utils/github-api.test.ts
+++ b/src/utils/github-api.test.ts
@@ -32,7 +32,7 @@ describe('getNotificationTypeIcon', () => {
       getNotificationTypeIcon(
         createSubjectMock({ type: 'CheckSuite', state: null }),
       ).displayName,
-    ).toBe('SyncIcon');
+    ).toBe('RocketIcon');
     expect(
       getNotificationTypeIcon(
         createSubjectMock({
diff --git a/src/utils/github-api.ts b/src/utils/github-api.ts
index f237040fd..f153d6ed1 100644
--- a/src/utils/github-api.ts
+++ b/src/utils/github-api.ts
@@ -20,7 +20,6 @@ import {
   RocketIcon,
   SkipIcon,
   StopIcon,
-  SyncIcon,
   TagIcon,
   XIcon,
 } from '@primer/octicons-react';
@@ -104,7 +103,7 @@ export function getNotificationTypeIcon(
         case 'success':
           return CheckIcon;
         default:
-          return SyncIcon;
+          return RocketIcon;
       }
     case 'Commit':
       return GitCommitIcon;

From 52dea875a9a179ca55d8356ff9321751e4ec9be6 Mon Sep 17 00:00:00 2001
From: Adam Setch <adam.setch@outlook.com>
Date: Mon, 11 Mar 2024 15:01:56 -0400
Subject: [PATCH 2/2] refactor(icons): replace cog icon with octicons prime
 version

---
 src/components/Sidebar.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/Sidebar.tsx b/src/components/Sidebar.tsx
index f7610e23b..6b831924a 100644
--- a/src/components/Sidebar.tsx
+++ b/src/components/Sidebar.tsx
@@ -71,7 +71,7 @@ export const Sidebar: React.FC = () => {
               }}
               aria-label="Refresh Notifications"
             >
-              <SyncIcon size={18} />
+              <SyncIcon size={16} />
             </button>
 
             <button