-
Notifications
You must be signed in to change notification settings - Fork 27.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show Reload full text as sub text #63187
Conversation
@@ -437,6 +402,67 @@ export class ExtensionEditor extends BaseEditor { | |||
return super.setInput(input, options, token); | |||
} | |||
|
|||
private setSubText(extension: IExtension, reloadAction: ReloadAction) { | |||
if (reloadAction.enabled) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sandy081 This if
never returns true. This is because the update of the "enabled" status on the ReloadAction is not a synchronous operation.
How do you suggest we tackle this? Other than adding a listener for when the ReloadAction changes its enabled status.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In such cases we listen to onDidChange event from the Action. See below
@@ -437,6 +402,67 @@ export class ExtensionEditor extends BaseEditor { | |||
return super.setInput(input, options, token); | |||
} | |||
|
|||
private setSubText(extension: IExtension, reloadAction: ReloadAction) { | |||
if (reloadAction.enabled) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In such cases we listen to onDidChange event from the Action. See below
e7bc117
to
fe83544
Compare
The |
This PR attempts to implement #60053 (comment)
Please reload Visual Studio Code to complete the .....ing of this extension.
This currently wont work due to https://github.com/Microsoft/vscode/pull/63187/files#r233713325