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

UI Themes: Should Have option/"old base grayscale" UI themes #25643

Closed
cleidigh opened this issue Apr 28, 2017 · 7 comments
Closed

UI Themes: Should Have option/"old base grayscale" UI themes #25643

cleidigh opened this issue Apr 28, 2017 · 7 comments
Assignees
Labels
feature-request Request for new features or functionality themes Color theme issues

Comments

@cleidigh
Copy link
Contributor

in the process of working on the new UI themes, it seems to me we should give
the users the ability to either use the old grayscale UI for any theme (maybe they didn't like our color choices) or create new themes with the old default colors perhaps named flat UI...

maybe there is already an easy way to do this I don't know about

@bpasero
Copy link
Member

bpasero commented Apr 29, 2017

@cleidigh can you explain better what you actually want?

@bpasero bpasero removed their assignment Apr 29, 2017
@bpasero bpasero added feature-request Request for new features or functionality themes Color theme issues labels Apr 29, 2017
@cleidigh
Copy link
Contributor Author

cleidigh commented Apr 29, 2017

@bpasero

  • Colorized UI components were just added to most built-in themes
  • What does the user who liked the main theme (eg Abyss) AND the mainly grayscale original UI color scheme (I think it was pretty great) have to do to accomplish this?
  • a workbench option? Certainly they should not have to redefine the original colors

@bpasero
Copy link
Member

bpasero commented Apr 30, 2017

Hm, I am still not sure I understand what you mean, can you give an example? Do you want the currently set color to be filled in when a user changes settings for colors to make it easier to see what is set currently?

@cleidigh
Copy link
Contributor Author

cleidigh commented Apr 30, 2017

Issue #25633 from @aeschli is a great idea , just when I wanted to do a small tweak on a color - different issue

sorry I can't seem to explain myself well...

Existing Use Case:

  • VCS User version <=1.11 - Using built in theme "Original" Kimbie Dark - or Monokai,etc (left side screenshot)

  • User likes the "Original" theme including the UI colors (eg Gray ActivityBar, LightBlue StatusBar etc)

  • This user now can only use the "New" Kimbie Dark (the right side screenshot) colorized UI ActivityBar etc - @aeschli - absolutely no criticism of your great color set ;-) Users may hate my monokai colors

  • What does this user (or any other user using built-in themes with "Original" UI colors have to do
    to get back to the left side "Original"

  • Basically do we need to have the "Original" themes included or an option to use "Original" UI colors

  • could be as simple as single workbench option workbench.colors.useBaseVCS_UIColors

  • or publish "Original" set of workbench theme UI colors

Maybe everyone will love the new colors and this is not an issue BUT I do think the original UI base colors were/are very good. At a minimum user should be able to use these either as a base or with
the "Original" built-in themes without having to reverse engineer/set every color in settings.json

Hopefully that is a better explanation..
Also I know you are in Endgame so of course ignore this until May I just wanted to post the issue when I thought about it.
image

@bpasero
Copy link
Member

bpasero commented Apr 30, 2017

@cleidigh Would it not be possible to start a new theme based off an existing theme (our generator supports that) and just tweak some colors and publish this theme as new theme?

@cleidigh
Copy link
Contributor Author

@bpasero
I did not think that the generator would necessarily stay - certainly somebody can use that with one of the base themes that still uses the grayscale UI layer that on top of abyss, Monokai, or any other
and get back to grayscale colors - seems easy enough.

Question: is issue #23506 the location you want to continue tracking elements that are not themable?

I'm studying the theme code starting with selectBox (item focus color etc) to help out in May to help finish up the other elements if that's useful.

generator sounds good - consider this closed

@bpasero
Copy link
Member

bpasero commented May 1, 2017

Ok, yes the generator is the starting point for theme authors. Feel free to add more things to #23506, though at one point we might just decide to close that one an open individual issues. Also, #23506 should not turn into the bucket for anything that cannot be themed, I would only throw things in there which are clearly looking bad when not being themed and having a specific theme enabled.

@bpasero bpasero closed this as completed May 1, 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 themes Color theme issues
Projects
None yet
Development

No branches or pull requests

3 participants