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

Fix #20768: Don't seed search without selection #80477

Closed

Conversation

wkornewald
Copy link
Contributor

@wkornewald wkornewald commented Sep 6, 2019

Fixes #20768. This makes CMD/CTRL+F consistent with other editors/IDEs (actually, almost all software on all platforms).

Without any text selected, the current behavior (with seedSearchStringFromSelection enabled) is to search for whatever word is under the cursor. This is unexpected because all (>99%) other software simply keeps the search field unchanged, so you can continue searching. Try it in some JetBrains IDE, Eclipse, NetBeans, Sublime, Notepad++, Firefox or Chrome in an input field, MS Word, or any other popular software. Nobody does that except for VS Code, so it breaks common user expectations and makes a very common use-case annoying (i.e. continuing from the last search).

That's why I have fully removed the old behavior. If you want to search for the word under the cursor you can simply select it before pressing CMD/CTRL+F like in almost every other application.

@msftclas
Copy link

msftclas commented Sep 6, 2019

CLA assistant check
All CLA requirements met.

This makes CMD/CTRL+F consistent with other editors/IDEs.
@AndrewUldin
Copy link

Hello, could you please give any estimates when this pull request could be merged? It's been more then 2 years that people have issues with this unexpected behaviour and thanks to @wkornewald it's been fixed but seems nobody is interested in accepting changes made by pro-active community members.

@rebornix rebornix added this to the February 2020 milestone Jan 27, 2020
@alexdima alexdima removed their assignment Feb 4, 2020
@SacredSkull
Copy link

This has been sat idle for nearly six months now - can someone with the relevant permissions confirm why this is problematic or merge it?

@Tyriar Tyriar modified the milestones: February 2020, March 2020 Mar 10, 2020
@rebornix rebornix modified the milestones: March 2020, On Deck Apr 2, 2020
@anlandu
Copy link
Member

anlandu commented Apr 19, 2020

Is there any timeline for this being merged? This would be an incredible UX improvement and I do not want to disable editor.find.seedSearchStringFromSelection entirely. Also, ctrl+h performs as expected, which makes me think this is not a universal behavior choice.

@farzher
Copy link

farzher commented May 16, 2020

hello vs code wakeup and merge this. your stupid ctrl f behavior is so annoying

@ralphie02
Copy link

@rebornix Is there a timeline on this? Can you please let people know.
Thanks!

@rebornix rebornix added feature-request Request for new features or functionality editor-find Editor find operations labels Oct 8, 2020
@rebornix rebornix modified the milestones: October 2020, Backlog Oct 30, 2020
@yumeyao
Copy link

yumeyao commented Dec 2, 2020

Oct, Nov, Dec ...
zzzzzZZZ

@joaomoreno joaomoreno changed the base branch from master to main February 15, 2021 08:51
@AlexDev2020
Copy link

When will these changes appear in vs code?

@rebornix rebornix removed their assignment Oct 22, 2021
@wkornewald
Copy link
Contributor Author

Apparently already fixed.

@wkornewald wkornewald closed this Nov 15, 2021
@wkornewald wkornewald deleted the no-seed-search-without-selection branch November 15, 2021 16:03
@github-actions github-actions bot locked and limited conversation to collaborators Dec 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
editor-find Editor find operations feature-request Request for new features or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option for seeding find string from editor on ctrl + f