Skip to content
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

Better support of library and control metadata in the Toolbox for referenced packages #9922

Closed
tbolon opened this issue Sep 14, 2023 · 4 comments

Comments

@tbolon
Copy link
Contributor

tbolon commented Sep 14, 2023

Environment

VS version: Version 17.8.0 Preview 1.0

.NET version

multiples

Did this work in a previous version of Visual Studio and/or previous .NET release?

yes, at least the designer handled packages references better in some points, worse on others.

Issue description

For a detailed description of this issue, see my repository where I made experiments.

My goal was to track the inconsistencies beetween the differents ways the designer handle toolbox items in visual studio, depending on how you reference controls in your projects (project ref vs package ref) and who is responsible for populating the toolbox (out-of-proc designer vs in-proc designer).

I have found inconsistencies on both project ref and package ref, and wanted to submit here an issue about the package references.

You will find in the readme of the repo linked a table with a summary about what is different, and suggestions about what could be fixed/enhanced.

The main points are:

  • The toolbox populating service should better implement features already present in the legacy designer: use product name and description set on the custom control
  • The legacy designer could display custom icons set on controls on packages. It already read come metadatas from packages and was supposed to handle toolboxbitmap long ago based on what @mgoertz-msft said on the issue linked.

I would love to add some horsepower and help fixing these gaps between the different experiences. If I could help in any way...

Back story: I am starting to investigate how to migrate the UI of our massive internal codebase to winforms in .net 8, since almost everything else has already been migrated. So I am examinating how our projects will handle multi-targeting.

PS: I will certainly create another issue regarding project references, which are handled differently.

Steps to reproduce

Diagnostics

No response

@tbolon tbolon added the untriaged The team needs to look at this issue in the next triage label Sep 14, 2023
@tbolon
Copy link
Contributor Author

tbolon commented Sep 14, 2023

Will close for now, need to investigate further 😅

@tbolon tbolon closed this as not planned Won't fix, can't repro, duplicate, stale Sep 14, 2023
@ghost ghost removed the untriaged The team needs to look at this issue in the next triage label Sep 14, 2023
@ilCosmico
Copy link

Will close for now, need to investigate further 😅

Why did you close this?

@tbolon
Copy link
Contributor Author

tbolon commented Sep 26, 2023

Because in the case of packages, it's relatively well supported: icons are missing on the VS integrated designer (which I suppose is not really well supported, as control packages were added recently for compatibility reasons) ; and the new designer is only missing product title and description.

I think priority should be given to project references, which should benefit from metadata's support (icons) and custom designer.

@ilCosmico
Copy link

Are you suggesting that it should also be able to support VisualStudioToolsManifest.xml?

@ghost ghost locked as resolved and limited conversation to collaborators Oct 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants