Skip to content

Add select all and clear actions to the source selector - #5460

Merged
Gabriel Dufresne (GabrielDuf) merged 1 commit into
mainfrom
feature/source-selector-select-all
Sep 30, 2026
Merged

Gabriel Dufresne (GabrielDuf) merged 1 commit into
mainfrom
feature/source-selector-select-all

Conversation

@GabrielDuf

Copy link
Copy Markdown
Contributor

This pull request refactors the styling and logic for "hyperlink-style" filter buttons and enhances the SearchSourceSelector control with improved UI and bulk selection features. The main changes include moving and consolidating style definitions, updating the layout and structure of the flyout content, and adding "Select all" and "Clear selection" bulk actions.

Styling and UI improvements:

  • Moved the Button.filter-hyperlink style definitions from AbstractPackagesPage.axaml to the shared Styles.Common.axaml file, ensuring consistent appearance and reducing duplication. These styles provide transparent borders and subtle hover/pressed fills for hyperlink-like filter buttons. [1] [2]

Feature enhancements in SearchSourceSelector:

  • Refactored the flyout content in SearchSourceSelector.cs to use a root StackPanel with increased spacing and width, and separated the list of checkboxes into its own panel for better layout and readability.
  • Added a bulk actions row with "Select all" and "Clear selection" buttons above the list of package manager checkboxes, allowing users to quickly select or clear all options.
  • Implemented reusable helper methods (BuildBulkActions, BuildLinkButton, and SetAll) to support the new bulk action buttons and apply the hyperlink-style button appearance.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused UI and selection changes are consistent, localized, and introduce no identified correctness issues.

Review effort: Balanced
Findings: None

What changed in this PR

Adds bulk source-selection actions to the search source selector and centralizes their shared hyperlink-style appearance.

Changes:

  • Adds “Select all” and “Clear selection” actions.
  • Improves flyout layout and scrolling.
  • Moves reusable button styles into shared resources.
File Description
AbstractPackagesPage.axaml Removes page-local hyperlink styles.
SearchSourceSelector.cs Adds bulk actions and refactors flyout layout.
Styles.Common.axaml Defines the shared hyperlink button styles.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@randy-but-a-ro randy-but-a-ro Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🤖 Pull request was approved automatically: the AI review is complete and all its review threads are resolved. 🎉

Integration Details
{
	"deliveryId": "d2e3cc00-bcdb-11f1-947d-3550bd1d8887",
	"headSha": "c17e9b25dc5a98d0c73966838e98ca3ec16b2faa",
	"reviewer": "copilot-pull-request-reviewer[bot]"
}

@GabrielDuf
Gabriel Dufresne (GabrielDuf) merged commit 30ece24 into main Sep 30, 2026
6 checks passed
@GabrielDuf
Gabriel Dufresne (GabrielDuf) deleted the feature/source-selector-select-all branch September 30, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants