fix: updated ui elements with a pencil to indicate that branch is edi…

…table
snyk · Jul 15, 2024 · f757c5c · f757c5c
1 parent cd30e1b
commit f757c5c
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/media/pencil.svg → media/images/pencil.svg b/media/pencil.svg → media/images/pencil.svg
diff --git a/src/snyk/common/views/issueTreeProvider.ts b/src/snyk/common/views/issueTreeProvider.ts
@@ -119,7 +119,7 @@ export abstract class ProductIssueTreeProvider<T> extends AnalysisTreeNodeProvid
     if (deltaFindingsEnabled) {
       return new TreeNode({
         text: 'Base branch: main',
-        icon: NODE_ICONS.branch,
+        icon: NODE_ICONS.pencil,
         command: this.setBaseBranchCommand(),
       });
     }

diff --git a/src/snyk/common/views/treeNode.ts b/src/snyk/common/views/treeNode.ts
@@ -7,7 +7,7 @@ export interface INodeIcon {
   ['dark']: string;
 }
 
-type NODE_ICON_TYPE = 'critical' | 'high' | 'medium' | 'low' | 'error' | 'branch';
+type NODE_ICON_TYPE = 'critical' | 'high' | 'medium' | 'low' | 'error' | 'branch' | 'pencil';
 
 export const NODE_ICONS: { [key in NODE_ICON_TYPE]: INodeIcon } = {
   critical: {
@@ -34,6 +34,10 @@ export const NODE_ICONS: { [key in NODE_ICON_TYPE]: INodeIcon } = {
     light: path.join(__filename, '..', '..', '..', '..', '..', 'media', 'images', 'branch_icon.svg'),
     dark: path.join(__filename, '..', '..', '..', '..', '..', 'media', 'images', 'branch_icon.svg'),
   },
+  pencil: {
+    light: path.join(__filename, '..', '..', '..', '..', '..', 'media', 'images', 'branch_icon.svg'),
+    dark: path.join(__filename, '..', '..', '..', '..', '..', 'media', 'images', 'pencil.svg'),
+  },
 };
 
 export type InternalType = {

diff --git a/src/snyk/snykCode/views/securityIssueTreeProvider.ts b/src/snyk/snykCode/views/securityIssueTreeProvider.ts
@@ -42,13 +42,13 @@ export default class CodeSecurityIssueTreeProvider extends IssueTreeProvider {
   protected getIssueFoundText(nIssues: number, ignoredIssueCount: number): string {
     if (nIssues > 0) {
       let text;
-      
+
       if (nIssues === 1) {
         text = `${nIssues} vulnerability found by Snyk`;
       } else {
         text = `✋ ${nIssues} vulnerabilities found by Snyk`;
       }
-    
+
       const isIgnoresEnabled = configuration.getFeatureFlag(FEATURE_FLAGS.consistentIgnores);
       if (isIgnoresEnabled) {
         text += `, ${ignoredIssueCount} ignored`;