Skip to content

Commit 861eff6

Browse files
authored
Merge pull request microsoft#159666 from microsoft/tyriar/156864
Move all terminal contributions to run on restored phase
2 parents a9e28d5 + cfe061f commit 861eff6

File tree

3 files changed

+10
-7
lines changed

3 files changed

+10
-7
lines changed

src/vs/workbench/contrib/externalTerminal/electron-sandbox/externalTerminal.contribution.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,12 @@ import { KeybindingsRegistry, KeybindingWeight } from 'vs/platform/keybinding/co
1313
import { Schemas } from 'vs/base/common/network';
1414
import { IConfigurationRegistry, Extensions, ConfigurationScope } from 'vs/platform/configuration/common/configurationRegistry';
1515
import { Registry } from 'vs/platform/registry/common/platform';
16-
import { IWorkbenchContribution } from 'vs/workbench/common/contributions';
16+
import { IWorkbenchContribution, IWorkbenchContributionsRegistry, Extensions as WorkbenchExtensions } from 'vs/workbench/common/contributions';
1717
import { IExternalTerminalMainService } from 'vs/platform/externalTerminal/electron-sandbox/externalTerminalMainService';
1818
import { IConfigurationService } from 'vs/platform/configuration/common/configuration';
1919
import { TerminalContextKeys } from 'vs/workbench/contrib/terminal/common/terminalContextKey';
2020
import { IRemoteAuthorityResolverService } from 'vs/platform/remote/common/remoteAuthorityResolver';
21+
import { LifecyclePhase } from 'vs/workbench/services/lifecycle/common/lifecycle';
2122

2223
const OPEN_NATIVE_CONSOLE_COMMAND_ID = 'workbench.action.terminal.openNativeConsole';
2324
KeybindingsRegistry.registerCommandAndKeybindingRule({
@@ -131,3 +132,7 @@ export class ExternalTerminalContribution implements IWorkbenchContribution {
131132
});
132133
}
133134
}
135+
136+
// Register workbench contributions
137+
const workbenchRegistry = Registry.as<IWorkbenchContributionsRegistry>(WorkbenchExtensions.Workbench);
138+
workbenchRegistry.registerWorkbenchContribution(ExternalTerminalContribution, LifecyclePhase.Restored);

src/vs/workbench/contrib/terminal/browser/terminal.contribution.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@ CommandsRegistry.registerCommand({ id: quickAccessNavigatePreviousInTerminalPick
7777

7878
// Register workbench contributions
7979
const workbenchRegistry = Registry.as<IWorkbenchContributionsRegistry>(WorkbenchExtensions.Workbench);
80-
workbenchRegistry.registerWorkbenchContribution(TerminalMainContribution, LifecyclePhase.Starting);
81-
workbenchRegistry.registerWorkbenchContribution(RemoteTerminalBackendContribution, LifecyclePhase.Starting);
80+
workbenchRegistry.registerWorkbenchContribution(TerminalMainContribution, LifecyclePhase.Restored);
81+
workbenchRegistry.registerWorkbenchContribution(RemoteTerminalBackendContribution, LifecyclePhase.Restored);
8282

8383
// Register configurations
8484
registerTerminalPlatformConfiguration();

src/vs/workbench/contrib/terminal/electron-sandbox/terminal.contribution.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import { Registry } from 'vs/platform/registry/common/platform';
99
import { TerminalIpcChannels } from 'vs/platform/terminal/common/terminal';
1010
import { ILocalPtyService } from 'vs/platform/terminal/electron-sandbox/terminal';
1111
import { IWorkbenchContributionsRegistry, Extensions as WorkbenchExtensions } from 'vs/workbench/common/contributions';
12-
import { ExternalTerminalContribution } from 'vs/workbench/contrib/externalTerminal/electron-sandbox/externalTerminal.contribution';
1312
import { ITerminalProfileResolverService } from 'vs/workbench/contrib/terminal/common/terminal';
1413
import { TerminalNativeContribution } from 'vs/workbench/contrib/terminal/electron-sandbox/terminalNativeContribution';
1514
import { ElectronTerminalProfileResolverService } from 'vs/workbench/contrib/terminal/electron-sandbox/terminalProfileResolverService';
@@ -22,6 +21,5 @@ registerSingleton(ITerminalProfileResolverService, ElectronTerminalProfileResolv
2221

2322
// Register workbench contributions
2423
const workbenchRegistry = Registry.as<IWorkbenchContributionsRegistry>(WorkbenchExtensions.Workbench);
25-
workbenchRegistry.registerWorkbenchContribution(LocalTerminalBackendContribution, LifecyclePhase.Starting);
26-
workbenchRegistry.registerWorkbenchContribution(TerminalNativeContribution, LifecyclePhase.Ready);
27-
workbenchRegistry.registerWorkbenchContribution(ExternalTerminalContribution, LifecyclePhase.Ready);
24+
workbenchRegistry.registerWorkbenchContribution(LocalTerminalBackendContribution, LifecyclePhase.Restored);
25+
workbenchRegistry.registerWorkbenchContribution(TerminalNativeContribution, LifecyclePhase.Restored);

0 commit comments

Comments
 (0)