Cleanup modal property from proposed api

上级 119055de
...@@ -115,7 +115,7 @@ export class TypeScriptVersionManager extends Disposable { ...@@ -115,7 +115,7 @@ export class TypeScriptVersionManager extends Disposable {
description: version.displayName, description: version.displayName,
detail: version.pathLabel, detail: version.pathLabel,
run: async () => { run: async () => {
const trusted = await vscode.workspace.requestWorkspaceTrust({ modal: true }); const trusted = await vscode.workspace.requestWorkspaceTrust();
if (trusted) { if (trusted) {
await this.workspaceState.update(useWorkspaceTsdkStorageKey, true); await this.workspaceState.update(useWorkspaceTsdkStorageKey, true);
const tsConfig = vscode.workspace.getConfiguration('typescript'); const tsConfig = vscode.workspace.getConfiguration('typescript');
......
...@@ -3027,20 +3027,18 @@ declare module 'vscode' { ...@@ -3027,20 +3027,18 @@ declare module 'vscode' {
*/ */
export interface WorkspaceTrustRequestOptions { export interface WorkspaceTrustRequestOptions {
/** /**
* When true, a modal dialog will be used to request workspace trust. * Custom message describing the user action that requires workspace
* When false, a badge will be displayed on the settings gear activity bar item. * trust. If omitted, a generic message will be displayed in the workspace
* trust request dialog.
*/ */
readonly modal: boolean; readonly message?: string;
} }
export namespace workspace { export namespace workspace {
/** /**
* Prompt the user to chose whether to trust the current workspace * Prompt the user to chose whether to trust the current workspace
* @param options Optional object describing the properties of the * @param options Optional object describing the properties of the
* workspace trust request. Defaults to { modal: false } * workspace trust request.
* When using a non-modal request, the promise will return immediately.
* Any time trust is not given, it is recommended to use the
* `onDidGrantWorkspaceTrust` event to listen for trust changes.
*/ */
export function requestWorkspaceTrust(options?: WorkspaceTrustRequestOptions): Thenable<boolean | undefined>; export function requestWorkspaceTrust(options?: WorkspaceTrustRequestOptions): Thenable<boolean | undefined>;
} }
......
...@@ -563,8 +563,7 @@ export class ExtHostWorkspace implements ExtHostWorkspaceShape, IExtHostWorkspac ...@@ -563,8 +563,7 @@ export class ExtHostWorkspace implements ExtHostWorkspaceShape, IExtHostWorkspac
} }
requestWorkspaceTrust(options?: vscode.WorkspaceTrustRequestOptions): Promise<boolean | undefined> { requestWorkspaceTrust(options?: vscode.WorkspaceTrustRequestOptions): Promise<boolean | undefined> {
const promise = this._proxy.$requestWorkspaceTrust(options); return this._proxy.$requestWorkspaceTrust(options);
return options?.modal ? promise : Promise.resolve(this._trusted);
} }
$onDidGrantWorkspaceTrust(): void { $onDidGrantWorkspaceTrust(): void {
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册