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

Update VS Options add/remove package source icons to VS2022 styling #5375

Merged
merged 1 commit into from
Aug 26, 2023

Conversation

timheuer
Copy link
Contributor

Bug

Fixes: NuGet/Home#12840

Regression? No

Description

This updates only the imagelist16/32/64 base64 encoded image for up/down/cancel/add images used in Package Source UI in VS2022. Source of new images are from the VS2022 KnownMonikers for MoveUp, MoveDown, Add, Cancel and match visuals of these type of actions elsewhere in VS2022.

Before:
55f95066-71e1-4c34-8318-b73ff766583e

After:
image

PR Checklist

  • PR has a meaningful title

  • PR has a linked issue.

  • Described changes

  • Tests

    • Automated tests added
    • OR
    • Test exception
    • OR
    • N/A
  • Documentation

    • Documentation PR or issue filled
    • OR
    • N/A

@timheuer timheuer requested a review from a team as a code owner August 24, 2023 06:13
@ghost ghost added the Community PRs created by someone not in the NuGet team label Aug 24, 2023
@zivkan
Copy link
Member

zivkan commented Aug 24, 2023

Using the "KnownMonikers Explorer" extension, I see that VS already has icons for Add, Cancel, MoveUp, and MoveDown. You pointed out in the PR description that these icons already exist in VS as KnownMonikers. VS extensibility also have docs on how to use it in WinForms: https://learn.microsoft.com/en-us/visualstudio/extensibility/image-service-and-catalog?view=vs-2022#how-do-i-update-winforms-ui

So, should we delete these icons from NuGet's resx, and use VS's built-in icons?

@timheuer
Copy link
Contributor Author

So, should we delete these icons from NuGet's resx, and use VS's built-in icons?

Honestly probably, but also I PR'd probably the smallest impact change...just the base64, no code, no DPI change, etc. -- I'd imagine changing to other means would be more code and then y'all would question the value of the change. I felt this was simplest to get there.

Copy link
Contributor

@aortiz-msft aortiz-msft left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution.

Copy link
Contributor

@donnie-msft donnie-msft left a comment

Choose a reason for hiding this comment

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

Tried it out and looks good! Thanks for helping us out!
image

@donnie-msft donnie-msft changed the title Update PMUI add/cancel icons to VS2022 styling Update VS Options add/remove package source icons to VS2022 styling Aug 25, 2023
Copy link
Contributor

@donnie-msft donnie-msft left a comment

Choose a reason for hiding this comment

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

The issue this closes mentions arrow icons, which I'm not sure I see in my testing. Left a comment here NuGet/Home#12840 (comment)

Once that's clarified, I'll merge. Thanks again!

@donnie-msft donnie-msft enabled auto-merge (squash) August 26, 2023 03:38
@donnie-msft donnie-msft merged commit fd095ec into NuGet:dev Aug 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community PRs created by someone not in the NuGet team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VS Options add/remove package source icons aren't using VS2022 styling
4 participants