-
Notifications
You must be signed in to change notification settings - Fork 693
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
Conversation
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. |
There was a problem hiding this 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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this 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!
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:
After:
PR Checklist
PR has a meaningful title
PR has a linked issue.
Described changes
Tests
Documentation