Skip to content

Commit 302f422

Browse files
authored
Fix configuration status not appearing. (#10852)
* Fix configuration status not appearing. * Update changelog date.
1 parent 13893fb commit 302f422

File tree

3 files changed

+7
-11
lines changed

3 files changed

+7
-11
lines changed

Extension/CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# C/C++ for Visual Studio Code Changelog
22

3-
## Version 1.15.3: March 20, 2023
3+
## Version 1.15.3: March 24, 2023
44
### Bug Fix
55
* Fix handling of sccache and clcache. [#7616](https://github.com/microsoft/vscode-cpptools/issues/7616)
66
* Fix an undefined reference regression. [PR #10824](https://github.com/microsoft/vscode-cpptools/pull/10824)

Extension/src/LanguageServer/ui.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -355,18 +355,16 @@ export class OldUI implements UI {
355355
if (isCppPropertiesJson) {
356356
vscode.languages.setTextDocumentLanguage(activeEditor.document, "jsonc");
357357
}
358+
const isCppOrRelated: boolean = isCppPropertiesJson || util.isCppOrRelated(activeEditor.document);
358359

359360
// It's sometimes desirable to see the config and icons when making changes to files with C/C++-related content.
360361
// TODO: Check some "AlwaysShow" setting here.
361-
const showConfigureIntelliSenseButtonForActiveFile: boolean = (isCppPropertiesJson || util.isCppOrRelated(activeEditor.document))
362-
&& !!this.currentClient && this.currentClient.getShowConfigureIntelliSenseButton();
363-
this.ShowConfiguration = showConfigureIntelliSenseButtonForActiveFile ||
364-
(util.getWorkspaceIsCpp() &&
362+
this.ShowConfiguration = isCppOrRelated || (util.getWorkspaceIsCpp() &&
365363
(activeEditor.document.fileName.endsWith("tasks.json") ||
366364
activeEditor.document.fileName.endsWith("launch.json")));
367365

368366
if (this.showConfigureIntelliSenseButton) {
369-
if (showConfigureIntelliSenseButtonForActiveFile) {
367+
if (isCppOrRelated && !!this.currentClient && this.currentClient.getShowConfigureIntelliSenseButton()) {
370368
this.configureIntelliSenseStatusItem.show();
371369
if (!this.configureIntelliSenseTimeout) {
372370
this.configureIntelliSenseTimeout = setTimeout(() => {

Extension/src/LanguageServer/ui_new.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -460,18 +460,16 @@ export class NewUI implements UI {
460460
if (isCppPropertiesJson) {
461461
vscode.languages.setTextDocumentLanguage(activeEditor.document, "jsonc");
462462
}
463+
const isCppOrRelated: boolean = isCppPropertiesJson || util.isCppOrRelated(activeEditor.document);
463464

464465
// It's sometimes desirable to see the config and icons when making changes to files with C/C++-related content.
465466
// TODO: Check some "AlwaysShow" setting here.
466-
const showConfigureIntelliSenseButtonForActiveFile: boolean = (isCppPropertiesJson || util.isCppOrRelated(activeEditor.document))
467-
&& !!this.currentClient && this.currentClient.getShowConfigureIntelliSenseButton();
468-
this.ShowConfiguration = showConfigureIntelliSenseButtonForActiveFile ||
469-
(util.getWorkspaceIsCpp() &&
467+
this.ShowConfiguration = isCppOrRelated || (util.getWorkspaceIsCpp() &&
470468
(activeEditor.document.fileName.endsWith("tasks.json") ||
471469
activeEditor.document.fileName.endsWith("launch.json")));
472470

473471
if (this.showConfigureIntelliSenseButton) {
474-
if (showConfigureIntelliSenseButtonForActiveFile) {
472+
if (isCppOrRelated && !!this.currentClient && this.currentClient.getShowConfigureIntelliSenseButton()) {
475473
this.configureIntelliSenseStatusItem.show();
476474
if (!this.configureIntelliSenseTimeout) {
477475
this.configureIntelliSenseTimeout = setTimeout(() => {

0 commit comments

Comments
 (0)