Skip to content

Create region headings & Audit Sources is collapsed until needed in VS Options#7010

Merged
donnie-msft merged 9 commits intodevfrom
dev-donnie-msft-auditVsOptionsCollapse
Jan 3, 2026
Merged

Create region headings & Audit Sources is collapsed until needed in VS Options#7010
donnie-msft merged 9 commits intodevfrom
dev-donnie-msft-auditVsOptionsCollapse

Conversation

@donnie-msft
Copy link
Copy Markdown
Contributor

@donnie-msft donnie-msft commented Dec 15, 2025

Bug

Fixes: NuGet/Home#14665

Description

  • Introduces Checkbox, "Use separate sources for vulnerability audit (?) help link"
    image

    AuditSourceOptionsCheckbox.mp4
    • When Unchecked, it means there are no audit sources and reiterates that package sources are used for NuGet Audit.
    • When Checked, the Audit sources table is shown where audit sources can be configured.
    • The static message is now shown dynamically only when one or more audit source is in the table:

      Remove all audit sources to revert to using package sources for vulnerability data.

    image
  • Renames the VS Options page to "Sources" to be a more generic title for the screen representing package/audit/machine-wide sources.

  • Creates headings (subcategories):

    • Package sources
    • Audit Sources
      • (?) help link. this section does not yet exist so it will open to the top of the page.
    • Machine-wide Package Sources
      • (?) help link
      • New description. Completed UX String review and we settled on a concise string:

      Machine‑wide package sources are provisioned by Visual Studio workloads and can only be enabled or disabled.

image

PR Checklist

  • Meaningful title, helpful description and a linked NuGet/Home issue
  • Added tests The only behavior change is a Unified Settings visib
  • Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc. Audit Source options needs documentation Home#14687

@donnie-msft donnie-msft requested a review from a team as a code owner December 15, 2025 20:43
Comment thread src/NuGet.Clients/NuGet.Tools/VSPackage.resx
@donnie-msft donnie-msft force-pushed the dev-donnie-msft-auditVsOptionsCollapse branch from 89fb610 to 7cf462c Compare December 17, 2025 00:24
@donnie-msft donnie-msft requested a review from nkolev92 December 17, 2025 00:47
zivkan
zivkan previously approved these changes Dec 17, 2025
Copy link
Copy Markdown
Member

@zivkan zivkan left a comment

Choose a reason for hiding this comment

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

Personally, I like changing the parent "Package sources" to just "Sources", since it (now) has 3 child sources (package, audit, machine)

@dotnet-policy-service dotnet-policy-service bot added the Status:No recent activity PRs that have not had any recent activity and will be closed if the label is not removed label Dec 25, 2025
@dotnet-policy-service dotnet-policy-service bot removed the Status:No recent activity PRs that have not had any recent activity and will be closed if the label is not removed label Dec 29, 2025
@donnie-msft
Copy link
Copy Markdown
Contributor Author

Updated Machine-wide package source description after UX String review.

@donnie-msft donnie-msft force-pushed the dev-donnie-msft-auditVsOptionsCollapse branch from 7cf462c to 6c8aaf4 Compare December 29, 2025 23:12
@donnie-msft donnie-msft requested a review from zivkan December 29, 2025 23:12
@donnie-msft donnie-msft requested a review from jeffkl December 31, 2025 01:48
@donnie-msft donnie-msft merged commit 36a5b01 into dev Jan 3, 2026
17 of 18 checks passed
@donnie-msft donnie-msft deleted the dev-donnie-msft-auditVsOptionsCollapse branch January 3, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Audit Sources region is collapsed until needed in VS Options

4 participants