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

The Find/Replace window covers the content #11102

Closed
jkarr opened this issue Aug 29, 2016 · 2 comments
Closed

The Find/Replace window covers the content #11102

jkarr opened this issue Aug 29, 2016 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) editor-find Editor find operations ux User experience issues
Milestone

Comments

@jkarr
Copy link

jkarr commented Aug 29, 2016

  • VSCode Version: Code 1.4.0 (6276dcb, 2016-08-04T16:45:31.680Z)
  • OS Version: Windows_NT ia32 10.0.10586

Steps to Reproduce:

1.Open a file with lots of content
2.Press Ctrl+F or Ctrl+H and the content underneath the find/replace is hidden

@dbaeumer dbaeumer added editor editor-find Editor find operations labels Aug 30, 2016
@alexdima alexdima added ux User experience issues bug Issue identified by VS Code Team member as probable bug labels Aug 30, 2016
@alexdima alexdima added this to the Backlog milestone Aug 30, 2016
@cparker4486
Copy link

I think it's important to note that this issue is an actual problem when the content that's being covered is at the very top of a file. In this case you can't scroll up to reveal the hidden content. Your only recourse is to temporarily add empty lines at the top to push the content down.

My preferred fix would be to adopt Sublime Text's implementation: a small horizontal control that slides up from the bottom of the window.

I recognize that my preferred fix completely changes the UI which might be too much to accept for the dev team and some users. (Even though I think it would be a net positive change.) So here are some other possible fixes.

  1. Display the dialog in the same place but shrink the content area by a maximum of three lines. Line 1 would visually move down and lines at the bottom of the window would be pushed below the visible area.
  2. Move the dialog itself down, up to three lines, if it covers matches. If it still covers matches (on lines 4,5,6) then move the dialog one line at a time when stepping through the matches. Move the dialog progressively downward, up to three lines, when stepping forward. Move the dialog back up, up to three lines, when coming from below the dialog while stepping backward. In other words, push the dialog in the direction the user is moving through the matches.

Let's be creative and find a good solution.

Thank you for your consideration!

@rebornix
Copy link
Member

rebornix commented May 5, 2017

Let's track it in #5525 and solve it in May.

@rebornix rebornix closed this as completed May 5, 2017
@rebornix rebornix added the *duplicate Issue identified as a duplicate of another issue(s) label May 5, 2017
@alexdima alexdima modified the milestones: Backlog, May 2017 May 31, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) editor-find Editor find operations ux User experience issues
Projects
None yet
Development

No branches or pull requests

5 participants