Skip to content

Commit 49b6172

Browse files
d13eamodio
authored andcommitted
Removes welcome webview
1 parent 1701081 commit 49b6172

31 files changed

+6
-3747
lines changed

docs/telemetry-events.md

Lines changed: 2 additions & 28 deletions
Large diffs are not rendered by default.

package.json

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@
5454
"onWebviewPanel:gitlens.patchDetails",
5555
"onWebviewPanel:gitlens.settings",
5656
"onWebviewPanel:gitlens.timeline",
57-
"onWebviewPanel:gitlens.welcome",
5857
"onStartupFinished"
5958
],
6059
"capabilities": {
@@ -4898,13 +4897,6 @@
48984897
"title": "General",
48994898
"order": 0,
49004899
"properties": {
4901-
"gitlens.showWelcomeOnInstall": {
4902-
"type": "boolean",
4903-
"default": true,
4904-
"markdownDescription": "Specifies whether to show the Welcome experience on first install",
4905-
"scope": "window",
4906-
"order": 10
4907-
},
49084900
"gitlens.showWhatsNewAfterUpgrades": {
49094901
"type": "boolean",
49104902
"default": true,
@@ -6075,11 +6067,6 @@
60756067
"title": "Show Visual File History View",
60766068
"category": "GitLens"
60776069
},
6078-
{
6079-
"command": "gitlens.showWelcomePage",
6080-
"title": "Welcome",
6081-
"category": "GitLens"
6082-
},
60836070
{
60846071
"command": "gitlens.showWorktreesView",
60856072
"title": "Show Worktrees View",
@@ -13203,11 +13190,6 @@
1320313190
"when": "extension =~ /^eamodio.gitlens?$/ && extensionStatus == installed",
1320413191
"group": "9_gitlens@1"
1320513192
},
13206-
{
13207-
"command": "gitlens.showWelcomePage",
13208-
"when": "extension =~ /^eamodio.gitlens?$/ && extensionStatus == installed",
13209-
"group": "9_gitlens@2"
13210-
},
1321113193
{
1321213194
"command": "gitlens.showSettingsPage",
1321313195
"when": "extension =~ /^eamodio.gitlens?$/ && extensionStatus == installed",

src/config.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,6 @@ export interface Config {
203203
readonly showDetailsView: 'open' | 'selection' | false;
204204
};
205205
readonly remotes: RemotesConfig[] | null;
206-
readonly showWelcomeOnInstall: boolean;
207206
readonly showWhatsNewAfterUpgrades: boolean;
208207
readonly sortBranchesBy: BranchSorting;
209208
readonly sortContributorsBy: ContributorSorting;

src/constants.commands.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,6 @@ export const enum Commands {
220220
ShowTagsView = 'gitlens.showTagsView',
221221
ShowTimelinePage = 'gitlens.showTimelinePage',
222222
ShowTimelineView = 'gitlens.showTimelineView',
223-
ShowWelcomePage = 'gitlens.showWelcomePage',
224223
ShowWorktreesView = 'gitlens.showWorktreesView',
225224
ShowWorkspacesView = 'gitlens.showWorkspacesView',
226225
StashApply = 'gitlens.stashApply',

src/constants.telemetry.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -611,7 +611,6 @@ export type Sources =
611611
| 'scm-input'
612612
| 'subscription'
613613
| 'walkthrough'
614-
| 'welcome'
615614
| 'worktrees';
616615

617616
export type Source = {

src/constants.views.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ export type GroupableTreeViewTypes = Extract<
3535
>;
3636
export type GroupableTreeViewIds<T extends GroupableTreeViewTypes = GroupableTreeViewTypes> = TreeViewIds<T>;
3737

38-
export type WebviewTypes = 'graph' | 'patchDetails' | 'settings' | 'timeline' | 'welcome';
38+
export type WebviewTypes = 'graph' | 'patchDetails' | 'settings' | 'timeline';
3939
export type WebviewIds = `gitlens.${WebviewTypes}`;
4040

4141
export type WebviewViewTypes = 'commitDetails' | 'graph' | 'graphDetails' | 'home' | 'patchDetails' | 'timeline';

src/container.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ import { VslsController } from './vsls/vsls';
6060
import { RebaseEditorProvider } from './webviews/rebase/rebaseEditor';
6161
import { registerSettingsWebviewCommands, registerSettingsWebviewPanel } from './webviews/settings/registration';
6262
import { WebviewsController } from './webviews/webviewsController';
63-
import { registerWelcomeWebviewPanel } from './webviews/welcome/registration';
6463

6564
export type Environment = 'dev' | 'staging' | 'production';
6665

@@ -231,8 +230,6 @@ export class Container {
231230
this._disposables.push(settingsPanels);
232231
this._disposables.push(registerSettingsWebviewCommands(settingsPanels));
233232

234-
this._disposables.push(registerWelcomeWebviewPanel(webviews));
235-
236233
this._disposables.push(new ViewFileDecorationProvider());
237234

238235
const patchDetailsPanels = registerPatchDetailsWebviewPanel(webviews);

src/extension.ts

Lines changed: 2 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ export async function activate(context: ExtensionContext): Promise<GitLensApi |
186186
);
187187
}
188188

189-
void showWelcomeOrWhatsNew(container, gitlensVersion, prerelease, previousVersion);
189+
void showWhatsNew(container, gitlensVersion, prerelease, previousVersion);
190190

191191
void storage.store(prerelease ? 'preVersion' : 'version', gitlensVersion);
192192

@@ -316,7 +316,7 @@ function registerBuiltInActionRunners(container: Container): void {
316316
);
317317
}
318318

319-
async function showWelcomeOrWhatsNew(
319+
async function showWhatsNew(
320320
container: Container,
321321
version: string,
322322
prerelease: boolean,
@@ -325,30 +325,6 @@ async function showWelcomeOrWhatsNew(
325325
if (previousVersion == null) {
326326
Logger.log(`GitLens first-time install; window.focused=${window.state.focused}`);
327327

328-
if (configuration.get('showWelcomeOnInstall') === false) return;
329-
330-
if (window.state.focused) {
331-
await container.storage.delete('pendingWelcomeOnFocus');
332-
await executeCommand(Commands.ShowWelcomePage);
333-
} else {
334-
// Save pending on window getting focus
335-
await container.storage.store('pendingWelcomeOnFocus', true);
336-
const disposable = window.onDidChangeWindowState(e => {
337-
if (!e.focused) return;
338-
339-
disposable.dispose();
340-
341-
// If the window is now focused and we are pending the welcome, clear the pending state and show the welcome
342-
if (container.storage.get('pendingWelcomeOnFocus') === true) {
343-
void container.storage.delete('pendingWelcomeOnFocus');
344-
if (configuration.get('showWelcomeOnInstall')) {
345-
void executeCommand(Commands.ShowWelcomePage);
346-
}
347-
}
348-
});
349-
container.context.subscriptions.push(disposable);
350-
}
351-
352328
return;
353329
}
354330

src/telemetry/walkthroughStateProvider.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ export class WalkthroughStateProvider implements Disposable {
3030
private readonly walkthroughByTracking: Partial<Record<TrackedUsageKeys, WalkthroughContextKeys>> = {
3131
[`command:${Commands.PlusStartPreviewTrial}:executed`]: WalkthroughContextKeys.GettingStarted,
3232
[`command:${Commands.PlusReactivateProTrial}:executed`]: WalkthroughContextKeys.GettingStarted,
33-
[`command:${Commands.ShowWelcomePage}:executed`]: WalkthroughContextKeys.GettingStarted,
3433
[`command:${Commands.OpenWalkthrough}:executed`]: WalkthroughContextKeys.GettingStarted,
3534
[`command:${Commands.GetStarted}:executed`]: WalkthroughContextKeys.GettingStarted,
3635

src/webviews/apps/settings/settings.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ import { formatDate, setDefaultDateLocales } from '../shared/date';
1717
import { DOM } from '../shared/dom';
1818
// import { Snow } from '../shared/snow';
1919
import '../shared/components/feature-badge';
20-
import '../welcome/components/gitlens-logo';
20+
import '../shared/components/gitlens-logo';
2121

2222
const topOffset = 83;
2323
const offset = (new Date().getTimezoneOffset() / 60) * 100;

0 commit comments

Comments
 (0)