Skip to content
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

Search-In-Workspace: Add multiselect functionality #12331

Conversation

FernandoAscencio
Copy link
Contributor

@FernandoAscencio FernandoAscencio commented Mar 21, 2023

What it does

Closes #11874

Implements in Search-In-Workspace:

  • Multiselect search results.
  • delete keybinding dismiss selected results.
  • Ctrl+C keybinding to copy selected results.
  • Handling for action buttons: Replace and Dismiss
  • Replace action in context menu with keybinding

enter in replace-inputbox refreshes search results,
instead of on every key.

Issue11874Fix

How to test

  1. Run Theia
  2. Open Search tab
  3. Search for a word
  4. Dismiss one or several selected results with either inline button, context menu or keybinding.
  5. Copy one search result using context menu or keybinding.
  6. Replace the searched word of selected results using the inline button, context menu or keybinding.

The other context menu options have been made to work as close as possible to VS Code.

Review checklist

Reminder for reviewers

@FernandoAscencio
Copy link
Contributor Author

@colin-grant-work

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

The changes look good to me 👍

  • Selecting multiple elements in the tree widget work as expected
  • Using the shortcut or context menu entry deletes the selected elements

@msujew msujew added the search in workspace issues related to the search-in-workspace label Apr 3, 2023
@FernandoAscencio
Copy link
Contributor Author

FernandoAscencio commented Apr 4, 2023

Added handling for action buttons: "Replace" and Dismiss when selecting multiple results.

@FernandoAscencio FernandoAscencio force-pushed the fa/MultiselectWorkspaceSearch branch 2 times, most recently from 05cd46c to 829a1f2 Compare April 5, 2023 15:53
@FernandoAscencio FernandoAscencio changed the title Search-In-Workspace: Implement multiselect and delete keybind Search-In-Workspace: Add multiselect functionality Apr 5, 2023
@vince-fugnitto vince-fugnitto modified the milestone: 1.37.0 Apr 27, 2023
@tsmaeder
Copy link
Contributor

tsmaeder commented May 8, 2023

@msujew can we merge this one?

Closes eclipse-theia#11874

Implements in Search-In-Workspace:
- Multiselect search results.
- `delete` keybinding dismiss selected results.
- `Ctrl`+`C` keybinding to copy selected results.
- Handling for action buttons: `Replace` and `Dismiss`
- `Replace` action in context menu with keybinding

`enter` in replace-inputbox refreshes search results,
instead of on every key.

Signed-Off-By: FernandoAscencio <fernando.ascencio.cama@ericsson.com>
@vince-fugnitto vince-fugnitto force-pushed the fa/MultiselectWorkspaceSearch branch from 9f64f03 to ca5da30 Compare May 8, 2023 13:42
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@vince-fugnitto vince-fugnitto merged commit 18b2503 into eclipse-theia:master May 8, 2023
@vince-fugnitto vince-fugnitto deleted the fa/MultiselectWorkspaceSearch branch May 8, 2023 14:16
@vince-fugnitto vince-fugnitto added this to the 1.38.0 milestone Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
search in workspace issues related to the search-in-workspace
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Multiselect and delete in Search-in-Workspace
4 participants