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

Change purple status line to scheme color when no folder selected #13

Merged
merged 1 commit into from
Nov 24, 2019
Merged

Conversation

knoxknox
Copy link
Contributor

When VSCode opens w/o opened folder, status line renders incorrectly.
See related issue: #8

This PR fixes this situation, and switch color to background color of scheme.

How to test:

  1. Open this file locally

~/.vscode/extensions/ionutvmi.spacegray-vscode-1.2.3/themes/base16-ocean.dark.json

  1. Under colors section add next line

377: "statusBar.noFolderBackground": "#1c1f26"

  1. Save this file and open VSCode to see related changes

Alternative solution is to add these lines to your settings.json:

"workbench.colorCustomizations": {
  "statusBar.noFolderBackground" : "#1c1f26"
}

When VSCode opens w/o opened folder, status line renders incorrectly.
See related issue: #8

This PR fixes this situation, and switch color to background color of scheme.

How to test:
1. Open this file locally
> ~/.vscode/extensions/ionutvmi.spacegray-vscode-1.2.3/themes/base16-ocean.dark.json
2. Under `colors` section add next line
> 377: "statusBar.noFolderBackground": "#1c1f26"
3. Save this file and open VSCode to see related changes

Alternative solution is to add these lines to your settings.json:
```
"workbench.colorCustomizations": {
  "statusBar.noFolderBackground" : "#1c1f26"
}
```
@knoxknox
Copy link
Contributor Author

Screen Shot 2019-11-16 at 8 57 20 PM

@knoxknox
Copy link
Contributor Author

Hi @ionutvmi, could you review this change ?

@mihai-vlc mihai-vlc merged commit 19ddfd5 into mihai-vlc:master Nov 24, 2019
@mihai-vlc
Copy link
Owner

Thank you for the contribution !

@mihai-vlc
Copy link
Owner

Included in 1.2.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants