-
Notifications
You must be signed in to change notification settings - Fork 5k
Document changes to Octicons #2315
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
Comments
Here's our only reference to it: https://code.visualstudio.com/api/references/vscode-api#StatusBarItem
|
As far as I know it's only the Status Bar and Quick Pick (see GitLens > Show Commit Details). @bpasero, can you confirm? |
Reminder: need to reference to it in https://github.com/Microsoft/vscode-extension-samples/tree/master/quickinput-sample too, then. |
@misolori I think octicons can also be used in hovers, I believe the GitLens extension (@eamodio) is doing that. He might also know more locations from his usage. |
AFAIK, they can only be in the status bar and quick picks. I had to resort to unicode "icons" in the hovers 😉 |
Ah ok 👍 |
Once the new version of Octicons is merged into master (microsoft/vscode#65989) we're going to need proper documentation of these changes in the release notes.
We should also try to address #1094 and provide proper documentation for Octicons in our extensible docs. @octref, do you have any ideas of where this should be placed in our docs?
The text was updated successfully, but these errors were encountered: