microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
162.5k stars 28.65k forks source link

Color update: title, activity bar, status, tabs, panel #16001

Closed bpasero closed 7 years ago

bpasero commented 7 years ago

Changes:

Try Out:

Dark Theme screen shot 2016-11-24 at 07 56 23

Light Theme screen shot 2016-11-24 at 07 56 15

HC Theme screen shot 2016-11-24 at 07 56 28

kumarharsh commented 7 years ago

+1, although I personally liked the blue band at the bottom.

jrieken commented 7 years ago

:-1: for the light theme. too much grey

bpasero commented 7 years ago

Here is what works well for me

And here is what does not:

joaomoreno commented 7 years ago

I pretty much subscribe to @bpasero's opinion.

kumarharsh commented 7 years ago

Atom's light theme was really good if i remember correctly. More 'soothing'.

isidorn commented 7 years ago

Same as @bpasero with the exception that I would prefer the panel to be the background color of the editor due to:

Also instead of changing the status bar color we should make that configurable via themes. If we change the status bar we will piss of some happy users.

screen shot 2016-11-24 at 10 11 53

stevencl commented 7 years ago

I think removing the lines from the tabs is a good change and removes some subtle distractions, although I would like to see what it looks like with multiple tab groups, especially in horizontal layout.

I agree with the comments about the dark borders with the light theme but I wonder if the rest of the proposal should just be something that people can configure or set with a theme? For example, @isidorn makes a good case for why he would like the panel background to be the same colour as the editor background but personally I would like it to be different. Many other editors and IDEs have an integrated terminal UI that has a different background to the editor and I like the feeling of being somewhere 'different' when I switch between the terminal and the editor.

bpasero commented 7 years ago

@stevencl the purpose of the download links was so that you can play around with these changes 👍

stevencl commented 7 years ago

D'oh! Completely missed that.

I can confirm that I like the way that horizontal layout looks :-)

bgse commented 7 years ago

Dark theme looks pretty slick this way, and frankly I like the status bar change on the light theme as well, gives the editor a more polished and streamlined look if you ask me.

It deviates from the Visual Studio brand, but not by that much imho. The VS 'brand' thing for me (in this case) is primarily the activity bar showing the mode, not a specific color. Or to put it another way, the brand is in specific little details in functionality, not theming details.

Agree with @isidorn though that this should be open to theming, people can be very picky about stuff like this, honestly I'll probably be a bit bummed myself if the dark and light themes don't land in the next update exactly as shown now that I've seen it ;-)

bgashler1 commented 7 years ago

Thanks for your feedback on the light theme.

Here are some proposed revisions.

(@bpasero I updated the PPT deck I sent earlier with the new color values).

NEW light-01

vs. OLD light-01-nolines

I lightened up the colors for the Activity Bar, statusbar and titlebar so that the editor area draws the eyes more than anything else.

bpasero commented 7 years ago

@bgashler1 I am not seeing a new color for the activity bar background (and title, status) in light theme so I went ahead and copied it out of the deck. Please verify:

joaomoreno commented 7 years ago

@bgashler1 It does seem to get better, but there's still something off about the whole big picture. The more I think about it, the more I come to the conclusion that it's the status bar. It just doesn't feel right when it blends in with the rest.

Also, it's hard to get a feeling for it with the screenshots, builds would be much better. 👍

bpasero commented 7 years ago

I ran new builds with latest changes:

joaomoreno commented 7 years ago

Having tried out the light theme in those builds:

It does feel better than just looking at the screenshots though.

isidorn commented 7 years ago

Tried it out and overall I do not feel like this is an improvement of the current design. Some details:

Bottom line: changing the default theme will be a shock for a lot of users. Even in the case that we improve the looks which IMHO is not the case now. Due to that we should just leave the default as it is and invest in making the workbench themable.

bgashler1 commented 7 years ago

Per our discussion today, we will not modify the default themes, but we will ship these modifications as UI themes later on and monitor feedback. See #16245