Fix microsoft#20920 - hide search results count if results go to 0

roblourens · roblourens · commit 82f11708deb8 · 2017-02-20T16:12:33.000-07:00
diff --git a/src/vs/workbench/parts/search/browser/searchViewlet.ts b/src/vs/workbench/parts/search/browser/searchViewlet.ts
@@ -1176,13 +1176,15 @@ export class SearchViewlet extends Viewlet {
 
 	private updateSearchResultCount(): void {
 		const fileCount = this.viewModel.searchResult.fileCount();
+		const msgWasHidden = this.messages.isHidden();
 		if (fileCount > 0) {
-			const msgWasHidden = this.messages.isHidden();
 			const div = this.clearMessage();
 			$(div).p({ text: this.buildResultCountMessage(this.viewModel.searchResult.count(), fileCount) });
 			if (msgWasHidden) {
 				this.reLayout();
 			}
+		} else if (!msgWasHidden) {
+			this.messages.hide();
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1176,13 +1176,15 @@ export class SearchViewlet extends Viewlet {`
`1176`	`1176`
`1177`	`1177`	`private updateSearchResultCount(): void {`
`1178`	`1178`	`const fileCount = this.viewModel.searchResult.fileCount();`
	`1179`	`+ const msgWasHidden = this.messages.isHidden();`
`1179`	`1180`	`if (fileCount > 0) {`
`1180`		`- const msgWasHidden = this.messages.isHidden();`
`1181`	`1181`	`const div = this.clearMessage();`
`1182`	`1182`	`$(div).p({ text: this.buildResultCountMessage(this.viewModel.searchResult.count(), fileCount) });`
`1183`	`1183`	`if (msgWasHidden) {`
`1184`	`1184`	`this.reLayout();`
`1185`	`1185`	`}`
	`1186`	`+ } else if (!msgWasHidden) {`
	`1187`	`+ this.messages.hide();`
`1186`	`1188`	`}`
`1187`	`1189`	`}`
`1188`	`1190`