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

Implement Find All command & button #33829

Closed
wants to merge 3 commits into from

Conversation

nickguletskii
Copy link

Description

Implements a Find All command and button which selects and adds multicursors to all matches in the file. This functionality is available in Sublime and I was a bit jealous, so I've decided I would implement it for VS Code. I understand that this contains a change to the UI, so I won't be upset if the UX team rejects this pull request, as I haven't discussed this feature with anyone before implementing it.

Usage examples

Simple editing using multi-cursors:
find-all-simple

Extracting matches:
find-all-extract-text

Further info

This pull request contains a placeholder icon I hacked up from other icons in VS Code's source, and it probably doesn't really match VS Code's search iconography.
This pull request doesn't contain any localisations.

@msftclas
Copy link

msftclas commented Sep 5, 2017

@nickguletskii,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@msftclas
Copy link

msftclas commented Sep 5, 2017

@nickguletskii, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, Microsoft Pull Request Bot

@nickguletskii
Copy link
Author

nickguletskii commented Sep 5, 2017

Turns out Github's issue search sucks and there's already a pull request for this: #7559

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants