Remove Legacy VS Options UI: WinForms & WPF View, Models, and Resources #6531
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bug
Fixes: Decommission Legacy VS Options NuGet Settings · Issue #14398 · NuGet/Home
Fixes: Remove copied WatermarkedTextbox styles once keys are available from VS Platform · Issue #12163 · NuGet/Home
Description
Given the following pages have been onboarded to Unified Settings:
By my calculations, the onboarding to Unified Settings has let me remove a net of 2,981 lines of src/ code (ignoring tests) from the NuGet.Client repo. The onboarding PRs added 2,631 lines of src/ code, and this PR removes 5,612 src/ lines.
This PR removes UI and supporting assets such as:
<UseWinForms>true</UseWinForms>
from NuGet.PackageManagement.UI.csproj.pkgdef
declaration which was used when both USX & Legacy options were in place.src/NuGet.Clients/NuGet.PackageManagement.UI/Options
Refactoring
OptionsPageActivator
for menu command handling.For example, Tools -> NuGet Package Manager -> NuGet Package Manager Settings and Package Manager Console's settings icon button.
Commented about the relationship between the legacy page GUIDs between USX's
registration.json
and NuGet'sOptionsPageActivator
.PR Checklist
Added tests