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 statusbar button #117
Conversation
src/Config.ts
Outdated
@@ -96,4 +96,8 @@ export class Config { | |||
public static get getMount(): Array<Array<string>> { | |||
return Config.getSettings<Array<Array<string>>>('mount'); | |||
} | |||
|
|||
public static get getShowOnStatusbar(): boolean { | |||
return Config.getSettings<boolean>('showOnStatusbar') || false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
false is meaningless here. Because default value set on settings. So it always get true
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but please fix the eslint errors:
https://travis-ci.org/ritwickdey/vscode-live-server/builds/375844209#L895-L896
Sorry for that, it wasn't picking up the spacing issue when i ran the linter locally. Should be right now :) |
Thanks a lot for the PR 💟 |
Maybe I haven't done this correctly because I'm a bit of a newbie to vscode extensions, and to contributing to code in general.
I just thought there should be an option to hide the "Go Live" button in the statusbar, I can't disable the extension because it is dependency for Live Sass Compiler, but it's a button I don't need because in most workspaces I'm not using it directly.
I thought I'd try doing it myself...