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

Option to hide the status bar implemented #6942

Merged
merged 1 commit into from
May 31, 2016

Conversation

kisstkondoros
Copy link
Contributor

This fixes #6356

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @isidorn and @bpasero to be potential reviewers

@msftclas
Copy link

Hi @kisstkondoros, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, MSBOT;

@bpasero
Copy link
Member

bpasero commented May 30, 2016

@kisstkondoros thanks 👍

@bpasero
Copy link
Member

bpasero commented May 30, 2016

@kisstkondoros can you merge this PR with master and resolve the conflicts? also I noticed a bug: you do not seem to read the status bar visibility setting on startup here: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/browser/workbench.ts#L412

@bpasero bpasero modified the milestones: June 2016, Backlog May 30, 2016
@kisstkondoros
Copy link
Contributor Author

@bpasero Will do it, thanks for spotting the issue 👍

@kisstkondoros
Copy link
Contributor Author

@bpasero I've rebased the branch 👍

@bpasero bpasero merged commit baa7ece into microsoft:master May 31, 2016
@kisstkondoros kisstkondoros deleted the hidestatusbar branch February 25, 2017 20:12
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option to hide the status bar
4 participants