From 8ad4b37635c12f6a389c7ea2229ad1156c083909 Mon Sep 17 00:00:00 2001 From: "sheche@microsoft.com" Date: Tue, 16 Apr 2019 20:49:46 +0800 Subject: [PATCH] Fix the bug that status bar failed to honor the setting --- src/statusbar/LeetCodeStatusBarItem.ts | 3 +-- src/statusbar/leetCodeStatusBarController.ts | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/statusbar/LeetCodeStatusBarItem.ts b/src/statusbar/LeetCodeStatusBarItem.ts index 749bbfac..5ca9bde6 100644 --- a/src/statusbar/LeetCodeStatusBarItem.ts +++ b/src/statusbar/LeetCodeStatusBarItem.ts @@ -16,11 +16,10 @@ export class LeetCodeStatusBarItem implements vscode.Disposable { switch (status) { case UserStatus.SignedIn: this.statusBarItem.text = `LeetCode: ${user}`; - this.statusBarItem.show(); break; case UserStatus.SignedOut: default: - this.statusBarItem.hide(); + this.statusBarItem.text = ""; break; } } diff --git a/src/statusbar/leetCodeStatusBarController.ts b/src/statusbar/leetCodeStatusBarController.ts index d6058010..6668a084 100644 --- a/src/statusbar/leetCodeStatusBarController.ts +++ b/src/statusbar/leetCodeStatusBarController.ts @@ -39,7 +39,7 @@ class LeetCodeStatusBarController implements Disposable { private isStatusBarEnabled(): boolean { const configuration: WorkspaceConfiguration = workspace.getConfiguration(); - return configuration.get("leetcode.enableStatusBar", false); + return configuration.get("leetcode.enableStatusBar", true); } }