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

Poor UI for Search/Replace #190835

Closed
FastieSystems opened this issue Aug 20, 2023 · 3 comments
Closed

Poor UI for Search/Replace #190835

FastieSystems opened this issue Aug 20, 2023 · 3 comments
Assignees
Labels
editor-find Editor find operations under-discussion Issue is under discussion for relevance, priority, approach

Comments

@FastieSystems
Copy link

Type: Feature Request

Search/Replace has a poor UI, inconsistent behavior, and hidden features.

For reference, the best S/R UI I have used is from the discontinued product Expression Web 4. If modeled after EW4, VSC's S/R would be much better.

I present a few poor behaviors here.

  1. If a selection is made in code and Ctrl+F or Ctrl+H is pressed, the selected text is not always automatically inserted into the find field automatically. It should be.

  2. When using Replace, tabbing between the search and replace fields should highlight the entire field.

  3. I only recently came to the realization that VSC was remembering previous search and replace strings. That's because the UI has no intuitive controls for moving between them. Instead of choosing specific keystrokes to move up and down, the up- and down-arrows were overloaded, a poor practice. EW4 provides a dropdown for both the S&R fields that displays the last 15 strings for each.

You can examine EW4's S/R UI by downloading EW4 from https://www.oldergeeks.com/downloads/file.php?id=3571 or finding it at its last resting place on the WayBack machine.

Here is what the EW4 dialog looks like.

EW4 Search and Replace Dialog
EW4 Search and Replace Dropdown

VS Code version: Code 1.81.1 (6c3e3db, 2023-08-09T22:22:42.175Z)
OS version: Windows_NT x64 10.0.19045
Modes:

@goyalyashpal
Copy link

whst more, triggering "find/replace in selection" is also an unwieldy path

  1. select whole text
  2. ctri + f (then enter iguess? will confirm tomorrow)
  3. press alt + l
  4. presh ctrl + f again
  5. type text
  6. press ctrl + h
  7. ... proceed

@rebornix rebornix added the editor-find Editor find operations label Aug 23, 2023
@rebornix rebornix added the under-discussion Issue is under discussion for relevance, priority, approach label Dec 6, 2023
@amunger
Copy link
Contributor

amunger commented Dec 14, 2023

the selected text is not always automatically inserted into the find field automatically

check your editor.find.seedSearchStringFromSelection setting. If setting that to always doesn't always seed the find widget, please open a bug.

Dropdown for search history is covered in #23099

And here is a related request to revamp the UI #116827

@FastieSystems
Copy link
Author

New information after experimentation: If the search text selected for find/replace contains a newline, the text will not be inserted into the find field. Otherwise, it will.

@github-actions github-actions bot locked and limited conversation to collaborators Jan 28, 2024
@aiday-mar aiday-mar added this to the December / January 2024 milestone Feb 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
editor-find Editor find operations under-discussion Issue is under discussion for relevance, priority, approach
Projects
None yet
Development

No branches or pull requests

5 participants