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

Wrong input box color in dark theme #7648

Closed
isidorn opened this issue Jun 14, 2016 · 5 comments
Closed

Wrong input box color in dark theme #7648

isidorn opened this issue Jun 14, 2016 · 5 comments
Assignees
Labels
*out-of-scope Posted issue is not in scope of VS Code themes Color theme issues ux User experience issues
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Jun 14, 2016

In light theme for input boxes we use the same background color as the one used for editor, however in dark theme that is not the case.
This becomes particularly visible when tabs are enabled as the quick open input has the same color as the empty after the tabs - looks confusing and this is the main reason why tabs visually look much better in the light theme than in the dark theme imho.

I can look into changing all the input box background colors but it is a big change and it requires UX blessing - so we can discuss it today in the ux meeting.

fyi @bpasero

screen shot 2016-06-14 at 11 41 58

@isidorn isidorn added ux User experience issues workbench-tabs VS Code editor tab issues labels Jun 14, 2016
@bpasero
Copy link
Member

bpasero commented Jun 14, 2016

+1 for making all inputs the same colour as the editor.

/cc @jrieken @alexandrudima for owners of input fields in the editor (find, rename).

@isidorn
Copy link
Contributor Author

isidorn commented Jun 15, 2016

Assigning to @bgashler1 to experiment with colors as we have agreed on our UX meeting and if we come to a conclusion give the issue back to me to implement it across vscode.

@stevencl
Copy link
Member

We discussed and agreed that we should look at the colours used throughout the product now that we have tabs enabled. @bgashler1 - this is something you and I should talk about.

This was referenced Jun 21, 2016
@bgashler1
Copy link
Contributor

bgashler1 commented Jun 28, 2016

@isidorn While Steven and I collaborate holistically on evaluating our colors, iconography, etc, I think the change in this PR #8238 may dampen the problem somewhat (below).

You'll notice that the tab well is now the same color as the viewlet which—aside from being more hierarchically correct color-wise—has the benefit of being less confusing with the command palette's input box.

screen shot 2016-06-27 at 9 51 13 pm

I'll move this to the backlog for now and we can keep this on our radar.

@bgashler1 bgashler1 modified the milestones: Backlog, June 2016 Jun 28, 2016
@bpasero bpasero removed the workbench-tabs VS Code editor tab issues label Jul 6, 2016
@mjbvz mjbvz assigned stevencl and unassigned bgashler1 Apr 11, 2017
@bpasero bpasero added the themes Color theme issues label Nov 11, 2017
@isidorn isidorn added the *out-of-scope Posted issue is not in scope of VS Code label Nov 20, 2017
@vscodebot
Copy link

vscodebot bot commented Nov 20, 2017

This feature request will not be considered in the next 6-12 months roadmap and has been closed to keep the number of issues we have to maintain manageable. See also our issue reporting guidelines.

Thanks for your understanding and happy coding!

@vscodebot vscodebot bot closed this as completed Nov 20, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*out-of-scope Posted issue is not in scope of VS Code themes Color theme issues ux User experience issues
Projects
None yet
Development

No branches or pull requests

4 participants