Skip to content

Remove file icon from Python file project items #436

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from
Closed

Conversation

Copilot
Copy link
Contributor

@Copilot 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 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 Copilot AI requested a review from eleanorjboyd May 20, 2025 21:26
Copilot finished work on behalf of eleanorjboyd May 20, 2025 21:26
@eleanorjboyd eleanorjboyd marked this pull request as ready for review May 20, 2025 21:49
@@ -232,7 +232,7 @@ export class ProjectItem implements ProjectTreeItem {
item.description = this.project.description;
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
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

Error loading sessions

Retrying...

Successfully merging this pull request may close these issues.

Inconsistent sizing, color, and usage of icons
4 participants