Skip to content

Commit

Permalink
Auto merge of #15707 - dfireBird:default-statusbar-action-config, r=l…
Browse files Browse the repository at this point in the history
…nicola

Add configuration for the default action of the status bar click action in VSCode

Fixes #15441

Please suggest any changes if necessary.
  • Loading branch information
bors committed Oct 4, 2023
2 parents ad73597 + ab091b7 commit b57658d
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 1 deletion.
13 changes: 13 additions & 0 deletions editors/code/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -371,6 +371,19 @@
],
"markdownDescription": "Problem matchers to use for `rust-analyzer.run` command, eg `[\"$rustc\", \"$rust-panic\"]`."
},
"rust-analyzer.statusBar.clickAction": {
"type": "string",
"enum": [
"stopServer",
"openLogs"
],
"enumDescriptions": [
"Stop Server",
"Open Logs"
],
"default": "openLogs",
"markdownDescription": "Action to run when clicking the extension status bar item."
},
"rust-analyzer.server.path": {
"type": [
"null",
Expand Down
4 changes: 4 additions & 0 deletions editors/code/src/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -329,6 +329,10 @@ export class Config {
get showDependenciesExplorer() {
return this.get<boolean>("showDependenciesExplorer");
}

get statusBarClickAction() {
return this.get<string>("statusBar.clickAction");
}
}

// the optional `cb?` parameter is meant to be used to add additional
Expand Down
6 changes: 5 additions & 1 deletion editors/code/src/ctx.ts
Original file line number Diff line number Diff line change
Expand Up @@ -400,7 +400,11 @@ export class Ctx {
statusBar.tooltip.appendText(status.message ?? "Ready");
statusBar.color = undefined;
statusBar.backgroundColor = undefined;
statusBar.command = "rust-analyzer.openLogs";
if (this.config.statusBarClickAction === "stopServer") {
statusBar.command = "rust-analyzer.stopServer";
} else {
statusBar.command = "rust-analyzer.openLogs";
}
this.dependencies?.refresh();
break;
case "warning":
Expand Down

0 comments on commit b57658d

Please sign in to comment.