Don't assume uncompleted category exists (#118250)

Jackson Kearl · web-flow · commit 6b7199812b4a · 2021-03-05T11:41:51.000-08:00
Fixes #118235
diff --git a/src/vs/workbench/contrib/welcome/gettingStarted/browser/gettingStarted.ts b/src/vs/workbench/contrib/welcome/gettingStarted/browser/gettingStarted.ts
@@ -613,7 +613,7 @@ export class GettingStartedPage extends EditorPane {
 			const progressAmount = assertIsDefined(progress.querySelector('.progress-bar-inner') as HTMLDivElement).style.width;
 			if (!toFocus && progressAmount !== '100%') { toFocus = assertIsDefined(progress.parentElement?.parentElement); }
 		});
-		(toFocus ?? assertIsDefined(this.container.querySelector('button.skip')) as HTMLButtonElement).focus();
+		(toFocus ?? assertIsDefined(this.container.querySelector('button.getting-started-category')) as HTMLButtonElement)?.focus();
 	}
 
 	private setSlide(toEnable: 'details' | 'categories') {

Original file line number	Diff line number	Diff line change
`@@ -613,7 +613,7 @@ export class GettingStartedPage extends EditorPane {`
`613`	`613`	`const progressAmount = assertIsDefined(progress.querySelector('.progress-bar-inner') as HTMLDivElement).style.width;`
`614`	`614`	`if (!toFocus && progressAmount !== '100%') { toFocus = assertIsDefined(progress.parentElement?.parentElement); }`
`615`	`615`	`});`
`616`		`- (toFocus ?? assertIsDefined(this.container.querySelector('button.skip')) as HTMLButtonElement).focus();`
	`616`	`+ (toFocus ?? assertIsDefined(this.container.querySelector('button.getting-started-category')) as HTMLButtonElement)?.focus();`
`617`	`617`	`}`
`618`	`618`
`619`	`619`	`private setSlide(toEnable: 'details' \| 'categories') {`