Applies performance tweak by @d13

nzaytsev · d13 · web-flow · commit 56e57119e4f2 · 2024-10-18T10:44:41.000+07:00
Co-authored-by: Keith Daulton &lt;keith.daulton@gitkraken.com&gt;
diff --git a/src/telemetry/walkthroughStateProvider.ts b/src/telemetry/walkthroughStateProvider.ts
@@ -122,8 +122,9 @@ export class WalkthroughStateProvider implements Disposable {
 	}
 
 	get progress() {
-		const doneValues = [...filter(this.state.values(), x => x)].length;
-		return doneValues / Object.keys(WalkthroughContextKeys).length;
+		const allValues = this.state.values();
+		const doneValues = [...filter(allValues, x => x)].length;
+		return doneValues / allValues.length;
 	}
 
 	dispose(): void {

Original file line number	Diff line number	Diff line change
`@@ -122,8 +122,9 @@ export class WalkthroughStateProvider implements Disposable {`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`get progress() {`
`125`		`- const doneValues = [...filter(this.state.values(), x => x)].length;`
`126`		`- return doneValues / Object.keys(WalkthroughContextKeys).length;`
	`125`	`+ const allValues = this.state.values();`
	`126`	`+ const doneValues = [...filter(allValues, x => x)].length;`
	`127`	`+ return doneValues / allValues.length;`
`127`	`128`	`}`
`128`	`129`
`129`	`130`	`dispose(): void {`