Skip to content

Conversation

@kzu
Copy link
Member

@kzu kzu commented Feb 27, 2023

Just like our PackagePath implies Pack=true, we should likewise automatically pack a None/Content item if it was used as the PackageIcon property.

We support this for relative paths within the project as well as linked files (in case users use the same icon across multiple projects).

This means that in the most common case (a .jpg/.png) alognside the project file, setting the PackageIcon property to that file will Just Work.

@kzu kzu added the enhancement New feature or request label Feb 27, 2023
@kzu kzu enabled auto-merge (rebase) February 27, 2023 21:20
Just like our PackagePath implies Pack=true, we should likewise automatically pack a None/Content item if it was used as the PackageIcon property.

We support this for relative paths within the project as well as linked files (in case users use the same icon across multiple projects).

This means that in the most common case (a .jpg/.png) alognside the project file, setting the PackageIcon property to that file will Just Work.
@kzu kzu force-pushed the dev/PackageIconInclude branch from 4cb987e to bdee1c9 Compare February 27, 2023 22:29
@kzu kzu merged commit 38ee8fc into main Feb 27, 2023
@kzu kzu deleted the dev/PackageIconInclude branch February 27, 2023 22:41
@devlooped devlooped locked and limited conversation to collaborators Nov 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants