The symbol link icon could be refreshed when file is changed

Author: ShuiRuTian

src/vs/workbench/browser/labels.ts

@@ -57,6 +57,11 @@ export interface IResourceLabelOptions extends IIconLabelValueOptions {
 	 * Will take the provided label as is and e.g. not override it for untitled files.
 	 */
 	readonly forceLabel?: boolean;
+
+	/**
+	 * Whether this resource is a symbolic link.
+	 */
+	readonly isSymbolicLink?: boolean;
 }
 
 export interface IFileLabelOptions extends IResourceLabelOptions {
@@ -456,6 +461,7 @@ class ResourceLabelWidget extends IconLabel {
 			}
 		}
 
+		const hasIsSymbolicLinkChanged = this.hasIsSymbolicLinkChanged(options);
 		const hasResourceChanged = this.hasResourceChanged(label);
 		const hasPathLabelChanged = hasResourceChanged || this.hasPathLabelChanged(label);
 		const hasFileKindChanged = this.hasFileKindChanged(options);
@@ -472,11 +478,18 @@ class ResourceLabelWidget extends IconLabel {
 		}
 
 		this.render({
-			updateIcon: hasResourceChanged || hasFileKindChanged,
-			updateDecoration: hasResourceChanged || hasFileKindChanged
+			updateIcon: hasResourceChanged || hasFileKindChanged || hasIsSymbolicLinkChanged,
+			updateDecoration: hasResourceChanged || hasFileKindChanged || hasIsSymbolicLinkChanged
 		});
 	}
 
+	private hasIsSymbolicLinkChanged(newOptions?: IResourceLabelOptions): boolean {
+		const isNewSymbolicLink = newOptions?.isSymbolicLink;
+		const isOldSymbolicLink = this.options?.isSymbolicLink;
+
+		return isNewSymbolicLink !== isOldSymbolicLink;
+	}
+
 	private hasFileKindChanged(newOptions?: IResourceLabelOptions): boolean {
 		const newFileKind = newOptions?.fileKind;
 		const oldFileKind = this.options?.fileKind;

src/vs/workbench/contrib/files/browser/views/explorerDecorationsProvider.ts

@@ -13,6 +13,7 @@ import { DisposableStore } from 'vs/base/common/lifecycle';
 import { explorerRootErrorEmitter } from 'vs/workbench/contrib/files/browser/views/explorerViewer';
 import { ExplorerItem } from 'vs/workbench/contrib/files/common/explorerModel';
 import { IExplorerService } from 'vs/workbench/contrib/files/browser/files';
+import { IFileService } from 'vs/platform/files/common/files';
 
 export function provideDecorations(fileStat: ExplorerItem): IDecorationData | undefined {
 	if (fileStat.isRoot && fileStat.isError) {
@@ -50,7 +51,8 @@ export class ExplorerDecorationsProvider implements IDecorationsProvider {
 
 	constructor(
 		@IExplorerService private explorerService: IExplorerService,
-		@IWorkspaceContextService contextService: IWorkspaceContextService
+		@IWorkspaceContextService contextService: IWorkspaceContextService,
+		@IFileService fileService: IFileService,
 	) {
 		this.toDispose.add(this._onDidChange);
 		this.toDispose.add(contextService.onDidChangeWorkspaceFolders(e => {
@@ -59,6 +61,9 @@ export class ExplorerDecorationsProvider implements IDecorationsProvider {
 		this.toDispose.add(explorerRootErrorEmitter.event((resource => {
 			this._onDidChange.fire([resource]);
 		})));
+		this.toDispose.add(fileService.onDidFilesChange(e => {
+			this._onDidChange.fire([...e.rawAdded, ...e.rawUpdated]);
+		}));
 	}
 
 	get onDidChange(): Event<URI[]> {

src/vs/workbench/contrib/files/browser/views/explorerView.ts

@@ -719,7 +719,7 @@ export class ExplorerView extends ViewPane implements IExplorerView {
 
 		await promise;
 		if (!this.decorationsProvider) {
-			this.decorationsProvider = new ExplorerDecorationsProvider(this.explorerService, this.contextService);
+			this.decorationsProvider = new ExplorerDecorationsProvider(this.explorerService, this.contextService, this.fileService);
 			this._register(this.decorationService.registerDecorationsProvider(this.decorationsProvider));
 		}
 	}

src/vs/workbench/contrib/files/browser/views/explorerViewer.ts

@@ -418,6 +418,7 @@ export class FilesRenderer implements ICompressibleTreeRenderer<ExplorerItem, Fu
 			fileKind: stat.isRoot ? FileKind.ROOT_FOLDER : stat.isDirectory ? FileKind.FOLDER : FileKind.FILE,
 			extraClasses: realignNestedChildren ? [...extraClasses, 'align-nest-icon-with-parent-icon'] : extraClasses,
 			fileDecorations: this.config.explorer.decorations,
+			isSymbolicLink: stat.isSymbolicLink,
 			matches: createMatches(filterData),
 			separator: this.labelService.getSeparator(stat.resource.scheme, stat.resource.authority),
 			domId