-
-
Notifications
You must be signed in to change notification settings - Fork 168
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
[FEATURE] - All Colors Settings to be Inside the Workbench Color Customizations. #644
Comments
Hi @pouyakary , This is not yet possible, because there is no API available to extract the color from Themes, and I need the color in order ot create the icon on the fly. There is an issue in the VS Code repo (microsoft/vscode#32813) asking for that, but no ETA. I'm subscibed to that issue, so any news I will be notified. In the meantime, I'm closing this issue, Thanks for your undestanding. |
I recently read the source of GitHub's Pull Request Extension on how they did it and it was really convenient: https://github.com/microsoft/vscode-pull-request-github/blob/main/package.json#L2721-L2731 You define a set of colors inside your package.json, and also give them a default value. Then a theme developer gets to define colors for that color you have defined and thus they can also support your extension on top of the normal colors the vscode has. The user as well can change the colors and even set them for different themes they have (like me for example for dark and light themes) |
Yep, that's exacly how contributed colors works, and the Bookmarks extension uses it ( Line 641 in 04ab0ca
ThemeColor which is not useful to create the .svg file on the fly.
|
Hello, hope you're doing well.
Thanks for the amazing extension. I could not have wanted this more.
I have a color theme that has two variants, one for dark and one for light mode. It is so exciting that your extension lets me customize the colors, but I wished all the color settings were included in the
workbench.colorCustomizations
. If that were the case I could have done 2 things that could have been so cool for me.The first was to specify the colors in my theme, so all my users could have the customizations matched to their themes once they had the two installed together.
I could have specified dark and light variants. I switch to dark light based on the sun and so if you included the settings it could have been so meaningful for me.
You already have 3 settings that are included in the
workbench.colorCustomization
. Can you please also move the following there?:bookmarks.backgroundLineColor
bookmarks.gutterIconBorderColor
bookmarks.gutterIconFillColor
Again, this extension is such an amazing thing. I'm so happy that I found it today!
Thanks a lot !
The text was updated successfully, but these errors were encountered: