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

Rename new git.* decorator color settings #37331

Closed
mjbvz opened this issue Oct 31, 2017 · 9 comments
Closed

Rename new git.* decorator color settings #37331

mjbvz opened this issue Oct 31, 2017 · 9 comments
Assignees
Labels
file-decorations verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@mjbvz
Copy link
Contributor

mjbvz commented Oct 31, 2017

From #37159

I found the new git explorer color names, such as "git.untrackedForeground", unclear when first browsing through them. Perhaps include the word explorer or resource in the name to make it explicit where these colors apply? Something like, "git.untrackedResourceForeground" or git.explorer.untrackedForeground

@jrieken jrieken added this to the October 2017 milestone Nov 1, 2017
@jrieken
Copy link
Member

jrieken commented Nov 1, 2017

@aeschli can you help me

@aeschli
Copy link
Contributor

aeschli commented Nov 1, 2017

Maybe gitFileDecoration.untrackedResourceForeground.
'explorer' sounds too restrictive to me. We ended up using the file icons also in many places.

@bpasero ?

@aeschli
Copy link
Contributor

aeschli commented Nov 1, 2017

Ah, there's already resource. So maybe gitResourceDecoration.untrackedForeground

@jrieken
Copy link
Member

jrieken commented Nov 1, 2017

Or , gitDecoration.modifiedResourceForeground, gitDecoration.deletedResourceForeground, gitDecoration.untrackedResourceForeground, gitDecoration.ignoredResourceForeground, gitDecoration.conflictResourceForeground

@jrieken
Copy link
Member

jrieken commented Nov 1, 2017

or gitFileDecoration.untrackedForeground etc

@bpasero
Copy link
Member

bpasero commented Nov 1, 2017

@jrieken @aeschli are these neutral to the selected SCM provider, and if so should they be prefixed with scm instead of git?

I am fine with gitFileDecoration.<state><foreground> (minus the question around git vs scm)

I find ResourceDecoration a bit technical.

@jrieken
Copy link
Member

jrieken commented Nov 1, 2017

It's git only. We don't use fileDecoration but just decoration in settings therefore I'd favour gitDecoration.modifiedResourceForeground etc

@jrieken jrieken closed this as completed in fe3f6cc Nov 1, 2017
@mjbvz mjbvz added the verification-needed Verification of issue is requested label Nov 2, 2017
@bpasero bpasero added the verified Verification succeeded label Nov 3, 2017
@bpasero
Copy link
Member

bpasero commented Nov 3, 2017

@jrieken
Copy link
Member

jrieken commented Nov 3, 2017

It was already there ;-) https://github.com/Microsoft/vscode-docs/blob/vnext/docs/getstarted/theme-color-reference.md#git-colors

@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
file-decorations verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants