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

When showing proposals above current line: Show proposals bottom to top #27646

Closed
chrmarti opened this issue May 30, 2017 · 7 comments
Closed
Assignees
Labels
feature-request Request for new features or functionality *out-of-scope Posted issue is not in scope of VS Code suggest IntelliSense, Auto Complete under-discussion Issue is under discussion for relevance, priority, approach
Milestone

Comments

@chrmarti
Copy link
Collaborator

  • VSCode Version: Code - Insiders 1.13.0-insider (afe47dc, 2017-05-30T10:34:18.253Z)
  • OS Version: Darwin x64 15.6.0
  • Extensions: none

Testing #27315

When showing proposals above current line: To keep the selected item close to the cursor position, the proposals should be order bottom to top, such that, the selected one is at the bottom where the cursor is:
screen shot 2017-05-30 at 2 19 38 pm

@ramya-rao-a
Copy link
Contributor

This is existing behavior, but I agree. I think so too.

Will look into what can be done here, but not for the May milestone.

@ramya-rao-a ramya-rao-a added feature-request Request for new features or functionality suggest IntelliSense, Auto Complete labels May 30, 2017
@ramya-rao-a ramya-rao-a added this to the Backlog milestone May 30, 2017
@ramya-rao-a
Copy link
Contributor

@jrieken What do you think of @chrmarti's proposal?

Though initially I thought it would be nice to have the selected item close to the cursor, the change in order can confuse users who would expect alphabetically sorted items (most of the time)

Another idea is instead of changing the order, show selected item and n items before it instead of showing the selected item and n items after it. This way, the order stays, and the selected item is closer to the cursor

@ramya-rao-a ramya-rao-a added the under-discussion Issue is under discussion for relevance, priority, approach label Dec 11, 2017
@jrieken
Copy link
Member

jrieken commented Dec 12, 2017

show selected item and n items before it instead of showing the selected item and n items after it.

Not sure what that means. Changing the sort order seems funky esp when I think about scrolling through the list

@ramya-rao-a
Copy link
Contributor

@jrieken Changing the sort order was what @chrmarti suggested.

What I am suggesting is to show

  • a
  • b
  • c
  • d (assume this is the current selection)
  • e (this is out of view)
  • f (this is out of view)

instead of

  • a (this is out of view)
  • b (this is out of view)
  • c (this is out of view)
  • d (this is the current selection)
  • e
  • f

So that the current selection is closer to the current line

@ramya-rao-a
Copy link
Contributor

The same issue has been brought up in the monaco-editor as well.
microsoft/monaco-editor#560

@jrieken
Copy link
Member

jrieken commented Dec 13, 2017

d (assume this is the current selection)

This is not very realistic as in most cases the first item is selected. A different item will only be selected when the prefix/language/suggestion has been selected before

@ramya-rao-a
Copy link
Contributor

Good point.

I'll close this issue then.

@ramya-rao-a ramya-rao-a added the *out-of-scope Posted issue is not in scope of VS Code label Dec 13, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality *out-of-scope Posted issue is not in scope of VS Code suggest IntelliSense, Auto Complete under-discussion Issue is under discussion for relevance, priority, approach
Projects
None yet
Development

No branches or pull requests

3 participants