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

Find and Replace dialog covers content #13362

Closed
cparker4486 opened this issue Oct 7, 2016 · 5 comments
Closed

Find and Replace dialog covers content #13362

cparker4486 opened this issue Oct 7, 2016 · 5 comments
Labels
*duplicate Issue identified as a duplicate of another issue(s)

Comments

@cparker4486
Copy link

  • VSCode Version: 1.5.3
  • OS Version: Win 10 1607

Steps to Reproduce:

  1. Have content as long as the window is wide in the first three lines of the window
  2. Display the Find and Replace dialog

image

In some cases, like mine above, this dialog covers the content and there's no way to change that except to make changes to the content itself by adding additional empty lines at the top.

Scrolling doesn't work and neither does changing the size of the window. The content will always be covered. Additionally, stepping through matches doesn't help because the dialog doesn't move out of the way when a match is found underneath it.

What's my preferred fix? I think a good solution is to model the control after Sublime Text's implementation. In ST's case, a single row dialog slides up from the bottom of the window. If it covers a line of text at the bottom, the content becomes scrollable which allows the user to scroll the content up. Similarly, stepping through the matches will make them visible because doing that repositions the content not the dialog.

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.)

Here are some other possible fixes I came up with.

  1. Display the dialog in the same place but shift the content itself down a maximum of three lines.
  2. Move the dialog 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 downard, 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.

There are certainly other options out there but I've spent too much time on this at this point. :)

Thank you for your consideration!

@Tyriar
Copy link
Member

Tyriar commented Oct 7, 2016

Duplicate #11102

@Tyriar Tyriar closed this as completed Oct 7, 2016
@cparker4486
Copy link
Author

All that effort... gone.

I even searched too. What a bummer.

@Tyriar
Copy link
Member

Tyriar commented Oct 7, 2016

@cparker4486 you should transfer the useful bits over to #11102

@cparker4486
Copy link
Author

Thanks for the suggestion. Would that be just a copy/paste?

@Tyriar
Copy link
Member

Tyriar commented Oct 7, 2016

Yep, thanks for the suggestions 😃

@Tyriar Tyriar added the *duplicate Issue identified as a duplicate of another issue(s) label Oct 7, 2016
@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
*duplicate Issue identified as a duplicate of another issue(s)
Projects
None yet
Development

No branches or pull requests

2 participants