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

More contrast to selected line in peek definition #12826

Closed
octref opened this issue Sep 27, 2016 · 9 comments
Closed

More contrast to selected line in peek definition #12826

octref opened this issue Sep 27, 2016 · 9 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug ux User experience issues

Comments

@octref
Copy link
Contributor

octref commented Sep 27, 2016

Currently it's too hard to tell which line is the one containing the symbol.

Really bad for scanning through.

image

@jrieken jrieken added the ux User experience issues label Sep 28, 2016
@jrieken
Copy link
Member

jrieken commented Sep 28, 2016

@bgashler1 please advice

@bgashler1
Copy link
Contributor

bgashler1 commented Sep 28, 2016

Just from a quick glance at this issue, I would expect this to be a line-highlight, since we're doing that for Search and Find/Replace. Similar concept, in my opinion. Thoughts?

screen shot 2016-09-28 at 4 47 47 pm

@octref
Copy link
Contributor Author

octref commented Sep 28, 2016

Would be best if the line has some shallow highlight like symbol selection:

image

While the symbol have a strong highlight, like the brown in search.

@bgashler1
Copy link
Contributor

@octref that also makes sense. Similar concept. Why are we using two different grays for these highlights? They seem like they should be the same.

@octref
Copy link
Contributor Author

octref commented Sep 29, 2016

@bgashler1 Not two grays. One line of gray background like our-editor, then the strong brown background on the keyword (editor in your case).

@bgashler1
Copy link
Contributor

bgashler1 commented Sep 29, 2016

@octref Sorry for the miscommunication, what I meant was that that the image I posted of a Search match line highlight has a very similar gray to the shallow gray in your image. But when I checked the color values of each, they differed. This is one of many color inconsistencies we have that we should eventually resolve.

@octref
Copy link
Contributor Author

octref commented Sep 29, 2016

@bgashler1 That I don't know. Indeed we could use more color consistency.

@jrieken jrieken added the bug Issue identified by VS Code Team member as probable bug label Apr 21, 2017
@jrieken
Copy link
Member

jrieken commented Apr 21, 2017

The problem is that some provider give us the word too highlight and some providers give us the whole definition range spanning multiple lines. Therefore we fold the highlight into a single position. Unsure what to do next...

@jrieken
Copy link
Member

jrieken commented Jun 28, 2017

This can be now configured with themes

@jrieken jrieken closed this as completed Jun 28, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 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 ux User experience issues
Projects
None yet
Development

No branches or pull requests

3 participants