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

Suggest widget looks weird today #5978

Closed
alexdima opened this issue Apr 29, 2016 · 6 comments
Closed

Suggest widget looks weird today #5978

alexdima opened this issue Apr 29, 2016 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@alexdima
Copy link
Member

Maybe the suggest widget could use a smaller line height than the editor?
Also, the text and icon are not vertically centered in each row.

image

@alexdima alexdima added this to the April 2016 milestone Apr 29, 2016
@alexdima
Copy link
Member Author

My settings:

    , "editor.fontFamily": "Hack"
    , "editor.renderWhitespace": true
    , "editor.fontSize": 16
    , "editor.lineHeight": 22

@joaomoreno joaomoreno added the bug Issue identified by VS Code Team member as probable bug label Apr 29, 2016
@joaomoreno
Copy link
Member

Yeah I did change something... Thanks for your settings, I'll take a look.

@joaomoreno
Copy link
Member

This comes from #2465, in which the suggest widget picks up on the line height and font size of the editor itself. I've pushed some changes to alleviate the line height difference in your case. Hope this solves it in the meantime. Would you rather have additional settings, suggest.fontSize and suggest.lineHeight?

I'll leave this issue open since the suggest does need some layout work to fix the vertical alignment of the icon et al.

@joaomoreno joaomoreno modified the milestones: May 2016, April 2016, Backlog Apr 29, 2016
@aeschli aeschli modified the milestones: April 2016, Backlog Apr 29, 2016
@joaomoreno
Copy link
Member

Reverted the changes from #2465, since it needs more work. 👍

@alexdima
Copy link
Member Author

I like your approach of deriving the font size and line height from the editor settings more than introducing separate settings.

@alexdima alexdima added the verified Verification succeeded label Apr 29, 2016
@joaomoreno
Copy link
Member

The problem is that the derivation is not so smooth across all platforms... gotta figure that out platform per platform. 👎

@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 verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants