Skip to content

Remove file icon from Python file project items#436

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/fix-168
Closed

Remove file icon from Python file project items#436
Copilot wants to merge 2 commits intomainfrom
copilot/fix-168

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 20, 2025

Overview

This PR removes the automatic file icon that was being applied to Python projects that are single .py files.

Before/After

Before: Python file projects showed a file icon
After: Both folder and file projects have no icons,

Fixes #168.

Co-authored-by: eleanorjboyd <26030610+eleanorjboyd@users.noreply.github.com>
Copilot AI changed the title [WIP] Inconsistent sizing, color, and usage of icons Remove file icon from Python file project items May 20, 2025
Copilot AI requested a review from eleanorjboyd May 20, 2025 21:26
@eleanorjboyd eleanorjboyd marked this pull request as ready for review May 20, 2025 21:49
item.tooltip = this.project.tooltip;
item.resourceUri = project.uri.fsPath.endsWith('.py') ? this.project.uri : undefined;
item.iconPath = this.project.iconPath ?? (project.uri.fsPath.endsWith('.py') ? ThemeIcon.File : undefined);
item.iconPath = this.project.iconPath;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here is the description of the resourceUri and iconPath:

                /**
		 * The icon path or {@link ThemeIcon} for the tree item.
		 * When `falsy`, {@link ThemeIcon.Folder Folder Theme Icon} is assigned, if item is collapsible otherwise {@link ThemeIcon.File File Theme Icon}.
		 * When a file or folder {@link ThemeIcon} is specified, icon is derived from the current file icon theme for the specified theme icon using {@link TreeItem.resourceUri resourceUri} (if provided).
		 */
		iconPath?: string | IconPath;

		/**
		 * The {@link Uri} of the resource representing this item.
		 *
		 * Will be used to derive the {@link TreeItem.label label}, when it is not provided.
		 * Will be used to derive the icon from current file icon theme, when {@link TreeItem.iconPath iconPath} has {@link ThemeIcon} value.
		 */
		resourceUri?: Uri;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inconsistent sizing, color, and usage of icons

4 participants