From 73483ca62db200fcc9ca28c0af958f4745e46ceb Mon Sep 17 00:00:00 2001 From: Wei Wu Date: Mon, 2 Dec 2024 10:53:08 +0800 Subject: [PATCH 1/2] feat: inherit VS Code file icons in SAS Content and Server --- .../ContentNavigator/ContentDataProvider.ts | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/client/src/components/ContentNavigator/ContentDataProvider.ts b/client/src/components/ContentNavigator/ContentDataProvider.ts index f6c1afe97..5cd1d11fb 100644 --- a/client/src/components/ContentNavigator/ContentDataProvider.ts +++ b/client/src/components/ContentNavigator/ContentDataProvider.ts @@ -18,7 +18,6 @@ import { TabInputText, TextDocument, TextDocumentContentProvider, - ThemeIcon, TreeDataProvider, TreeDragAndDropController, TreeItem, @@ -230,6 +229,7 @@ class ContentDataProvider iconPath: this.iconPathForItem(item), id: item.uid, label: item.name, + resourceUri: uri, }; } @@ -698,7 +698,7 @@ class ContentDataProvider private iconPathForItem( item: ContentItem, - ): ThemeIcon | { light: Uri; dark: Uri } { + ): undefined | { light: Uri; dark: Uri } { const isContainer = getIsContainer(item); let icon = ""; if (isContainer) { @@ -723,11 +723,6 @@ class ContentDataProvider icon = "folder"; break; } - } else { - const extension = item.name.split(".").pop().toLowerCase(); - if (extension === "sas") { - icon = "sasProgramFile"; - } } return icon !== "" @@ -738,7 +733,7 @@ class ContentDataProvider `icons/light/${icon}Light.svg`, ), } - : ThemeIcon.File; + : undefined; } } From df54981bfe90c97a3648043752a9622446403ecb Mon Sep 17 00:00:00 2001 From: Wei Wu Date: Mon, 2 Dec 2024 11:09:44 +0800 Subject: [PATCH 2/2] fix: unit test --- .../components/ContentNavigator/ContentDataProvider.test.ts | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/client/test/components/ContentNavigator/ContentDataProvider.test.ts b/client/test/components/ContentNavigator/ContentDataProvider.test.ts index 4caa8fc65..879eece50 100644 --- a/client/test/components/ContentNavigator/ContentDataProvider.test.ts +++ b/client/test/components/ContentNavigator/ContentDataProvider.test.ts @@ -3,7 +3,6 @@ import { DataTransferItem, FileStat, FileType, - ThemeIcon, TreeItem, Uri, authentication, @@ -175,7 +174,6 @@ describe("ContentDataProvider", async function () { const treeItem = await dataProvider.getTreeItem(contentItem); const uri = contentItem.vscUri; const expectedTreeItem: TreeItem = { - iconPath: ThemeIcon.File, id: "unique-id", label: "testFile", command: { @@ -183,6 +181,7 @@ describe("ContentDataProvider", async function () { arguments: [uri], title: "Open SAS File", }, + resourceUri: uri, }; expect(treeItem).to.deep.include(expectedTreeItem);