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

[themes] Put theme configuration to settings.json #7358

Closed
Tyriar opened this issue Jun 7, 2016 · 15 comments
Closed

[themes] Put theme configuration to settings.json #7358

Tyriar opened this issue Jun 7, 2016 · 15 comments
Assignees
Labels
feature-request Request for new features or functionality plan-item VS Code - planned item for upcoming themes Color theme issues
Milestone

Comments

@Tyriar
Copy link
Member

Tyriar commented Jun 7, 2016

Having a theme/currentTheme key that could be set in settings.json would be useful in automatically setting the theme in an automated setup. I'm assuming right now it's handled in local storage or something instead?

@Tyriar Tyriar added *question Issue represents a question, should be posted to StackOverflow (VS Code) themes Color theme issues labels Jun 7, 2016
@aeschli
Copy link
Contributor

aeschli commented Jun 8, 2016

Yes, part of your workbench settings that know about recently opened VSCode input, window sizes, cursor positions.
We could put that in the settings. Would we remove it from the File>Preferences menu then? We would loose the theme previewing. If we keep it we would have to ask the user of where to define the theme in user or workbench settings.

@aeschli aeschli added feature-request Request for new features or functionality and removed *question Issue represents a question, should be posted to StackOverflow (VS Code) labels Jun 8, 2016
@aeschli aeschli added this to the Backlog milestone Jun 8, 2016
@aeschli aeschli changed the title Why is the current theme handled outside of settings.json [themes] Put theme configuration to settings.json Jun 8, 2016
@Tyriar
Copy link
Member Author

Tyriar commented Jun 8, 2016

Couldn't the theme preview just live update the settings.json?

@aeschli
Copy link
Contributor

aeschli commented Jun 8, 2016

Which one? User or workspace settings?

@Tyriar
Copy link
Member Author

Tyriar commented Jun 9, 2016

@aeschli well it would be weird to have a workspace-associated theme, so user. Good point though that there would be a bit of weirdness there. In the end I think it would be good to be able to script your entire vscode setup, themes are the main missing piece in that now that extensions can be installed via the CLI.

@Tyriar Tyriar closed this as completed Jun 9, 2016
@Tyriar Tyriar reopened this Jun 9, 2016
@gerane
Copy link

gerane commented Jun 17, 2016

I say you keep the two synced. I'd like to have that ability.

@isvoboda
Copy link

@Tyriar actually there are situations when theme per workspace makes sense -- I am dealing with the reduced color pallet in the default light+ and dark+ themes which forced me to switch the theme every time I am working with different types of code i.e. workspace.

@siegebell
Copy link

When developing extensions, I often have two vscode instances running (sometimes four!) and it would be very helpful if I could quickly distinguish them by assigning each a different theme.

@Jackbennett
Copy link

To add some use cases, I'd like the set the theme per workstation for our environment. I specifically want to mandate a light theme because our users screenshot examples from VSCode and it'd save a lot of toner. But I appreciate that's quite specific. If you add printing support you might also want the ability to use a specific print theme.

Perhaps a workstation.json can be superseded by the settings.json which as currently is superseded by the workspace.json?

I think the quickest improvement would be to add the theme key to settings.json which takes precedence over storage.json. I currently create a fixed settings.json with a GPO to preset other things, it's easy enough for admins to copy across settings they want to keep.

@cheapsteak
Copy link

it would be weird to have a workspace-associated theme

I actually found this thread because I was trying to figure out how to get a workspace-associated theme. I'm quite frequently working on multiple projects and being able to associate a specific project's editor window with a particular color would be incredibly helpful

@steveinatorx
Copy link

this would be a killer feature.

@atmd83
Copy link

atmd83 commented Jan 26, 2017

just setting up a new machine after putting my settings.json in source control to maintain across boxes. Having the theme 'settable' from settings.json seems a really worthwhile feature (as well as the icon theme)

is automatically installing a theme from the market place if it isn't currently installed (and set in settings.json) too big a step?

@aeschli aeschli added the plan-item VS Code - planned item for upcoming label Feb 9, 2017
@aeschli aeschli modified the milestones: February 2017, Backlog Feb 9, 2017
@kieferrm kieferrm mentioned this issue Feb 9, 2017
38 tasks
@hashhar
Copy link
Contributor

hashhar commented Feb 9, 2017

Could it be implemented with an additional setting key called "saveThemeTo" with the values, "user", "workspace" to decide where to make changes by default during live preview?

Also, what are the scenarios when the "theme" setting would need to be auto-changed? Just when live-previewing I guess?

EDIT

Another mechanism can be changing the "theme" in user settings unless a "theme" has been specified in workspace settings.
If a theme has been specified in worskspace settings, update both when live-previewing?

The dirtiest but easiest way would be to obviously add two different menu items.

@aeschli
Copy link
Contributor

aeschli commented Feb 21, 2017

Theme configuration is now part of the settings:

  // Specifies the color theme used in the workbench.
  "workbench.colorTheme": "Default Dark+",

  // Specifies the icon theme used in the workbench.
  "workbench.iconTheme": null,

It can be configured as a user and also as a workspace setting.

@shanalikhan
Copy link

shanalikhan commented Feb 21, 2017

So i can get all the installed file icons and themes both from iterating in the extensions ?
(vscode.extension.all.forEach(e => e.packageJSON.contributes ....)

And set current via settings.json !

@aeschli
Copy link
Contributor

aeschli commented Feb 21, 2017

Yes, in the packageJSON you find all contributed themes & icon theme including the path to the theme file and icon theme file.

@aeschli aeschli closed this as completed Feb 21, 2017
@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
feature-request Request for new features or functionality plan-item VS Code - planned item for upcoming themes Color theme issues
Projects
None yet
Development

No branches or pull requests