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

Show Icons for Completion Kind #10056

Closed
wants to merge 9 commits into from
Closed

Conversation

Feel-ix-343
Copy link
Contributor

Takes influence from the icons here and adds completion kind icons to zed completions. (It looks beautiful!)

Preview with Rust-Analyzer on the Zed codebase:

zed_completion_kind

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Apr 1, 2024
@Feel-ix-343 Feel-ix-343 marked this pull request as ready for review April 1, 2024 21:56
@Feel-ix-343
Copy link
Contributor Author

And a future step could be to make this configurable by Zed themes

@Feel-ix-343
Copy link
Contributor Author

I will make the icons the same color as the text!

Copy link
Member

@maxdeviant maxdeviant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's unlikely we'll merge in this current state, as we would need to design all of these icons to match our current icon set.

@maxdeviant maxdeviant self-assigned this Apr 1, 2024
@Feel-ix-343
Copy link
Contributor Author

Okay. Are there plans for themes to configure icons?

@maxdeviant
Copy link
Member

Okay. Are there plans for themes to configure icons?

We are planning on adding the ability for extensions to provide icons, but no timeline on it yet. Right now our primary focus with extensions is making sure all of our current language servers can be expressed using the extension API.

Here's the tracking issue for icons in extensions: #8843

@filipwiech
Copy link

Maybe this PR could be repurposed and show text with a styled font (muted + colored depending on the completion type) instead of the icons for now? 🤔 Icons could then always be added later and made configurable (maybe the text should be configurable by the user/theme too). 🙂

@maxdeviant
Copy link
Member

Maybe this PR could be repurposed and show text with a styled font (muted + colored depending on the completion type) instead of the icons for now? 🤔 Icons could then always be added later and made configurable (maybe the text should be configurable by the user/theme too). 🙂

I think we’re a little pressed for real estate here to be showing words of variable lengths.

@filipwiech
Copy link

Maybe this PR could be repurposed and show text with a styled font (muted + colored depending on the completion type) instead of the icons for now? 🤔 Icons could then always be added later and made configurable (maybe the text should be configurable by the user/theme too). 🙂

I think we’re a little pressed for real estate here to be showing words of variable lengths.

That's understandable. 🙂 To be honest I wasn't really thinking about whole words, but more like letters, kind of similar to what for example IntelliJ does (m - method, f - field, v - variable, etc.), but perhaps it would be not clear enough. 👍

@jezikk
Copy link

jezikk commented Apr 2, 2024

XCode examples:
XCode

XCode2

@Feel-ix-343
Copy link
Contributor Author

XCode examples: XCode
XCode2

I like this to - it is technically another icon style though and I think either way the Zed team would want to redesign the icons; It could be quite simple if they are just characters, however.

@maxdeviant
Copy link
Member

I'm going to close this for now, as I think we'll want to involve our designer, @/iamnbutler, when he's back from paternity leave to make sure we settle on an approach that aligns with our design direction for Zed.

@maxdeviant maxdeviant closed this Apr 2, 2024
@Feel-ix-343
Copy link
Contributor Author

Hey!

@iamnbutler bump (no pressure)

@d1y
Copy link
Contributor

d1y commented Apr 25, 2024

@iamnbutler If you have time, can you take a look at this? ^:)

@retort00
Copy link

@iamnbutler bump 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Code completion icons
6 participants