Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 8, 2025

Sort and filter are conceptually different operations. Users expected sort controls separate from the filter dropdown, not nested within "More Filters...".

Changes

Moved comment sort from filter submenu to dedicated view title button:

  • Relocated commentSortSubmenu from viewFilterSubmenu to MenuId.ViewTitle
  • Changed icon from Codicon.history to Codicon.sortPrecedence
  • Updated menu group from '2_sort' to 'navigation' (order: 1)

Before: Filter dropdown → More Filters → Sort By
After: Dedicated sort button in view title bar (adjacent to filter)

Filter actions (Show Resolved/Unresolved) remain in viewFilterSubmenu as intended.

Original prompt

This section details on the original issue you should resolve

<issue_title>Break comment sort into a separate button outside of the filter input</issue_title>
<issue_description>Testing #226657

While filtering is certainly related to sorting, they're different things. I was not expecting sort actions to be there:

image

I see we have this icon:

image

But I was expecting one like this:

image

or

image
</issue_description>

Comments on the Issue (you are @copilot in this section)

@alexr00 This was inspired by the extensions view. @daviddossett is there a canonical way we do this? @alexr00 Makes sense, thank you for weighing in!

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Copilot AI changed the title [WIP] Break comment sort into a separate button Move comment sort to separate view title button Dec 8, 2025
Copilot AI requested a review from alexr00 December 8, 2025 11:42
Copilot finished work on behalf of alexr00 December 8, 2025 11:42
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.

Break comment sort into a separate button outside of the filter input

2 participants